Pedro Cattori

Results 43 comments of Pedro Cattori

In Remix, we already make properties that are union including `undefined` into optionals (see remix-run/remix#3766 and remix-run/remix#3879 ), so for what it's worth we think its a good way to...

Here's an example of a custom hook built on `useScrollSync`: ```tsx import React from 'react' import { useScrollSync } from 'react-scroll-sync' export const useTableLayout = ({thead, tbody}) => { useScrollSync([thead,...

> I like this direction very much! Do you want to drop components in favor of hooks and do a breaking change? Or do you want to support both? I...

> @pcattori I resolved almost all comments and awaiting for the final changes. Also, could you document the hook / update the documentation to reflect the new state? > >...

@okonet: I've implemented the `mode` parameter, added jsdoc for `useScrollSync`, and reworked the example into a markdown usage guide. I think we should be good to go with that! Let...

> @okonet, @pcattori Really appreciate your work, everyone. > but it seems like some type checking in TS are not passing. `useElement` is returning `CallbackRef` but it is not compatible...

@mcansh : let's do some solid testing on the `esbuild` upgrade before we merge this

I think this will also help for support with in-memory caches / databases / stores / etc... on the server in dev. Currently, any in-memory data gets cleared out when...

One workaround could be to only watch the assets manifest and see if the hash in there changes. If you need to watch for server builds, then you only need...

I'm also looking into a Node API for the Remix compiler so that you could call `watch` yourself with the appropriate callbacks (E.g. `onRebuildSuccess` or something). I think that would...