Nick Randall
Nick Randall
I would love to see this PR landed. Are you open to this change? Anything I can do to help get it past the finish line?
@devongovett I agree with @drewdecarme that this is an important feature. If I'm building a component library, I would like the bundled output to not require any more bundling so...
I'm also seeing this problem in a Remix app. Thanks for looking into it!
It would be really great if we could abstract the `isActive` logic into its own hook and then expose it so that it can be used in other components.
Still happening on 1.3.4
I am also seeing this issue. `pnpm exec rome` spits out the same error as above.
@benwis I'll give you access to my branch. I also don't care which PR gets merged so if it's easier to close then and re-open yours, we can do that.
One event that passive events are common for (that is not on the document) is touchstart events. React 18 always uses "passive" for touchstart events.
@dvj1988 any chance you'd be willing to publish your fork to NPM?
Also seeing this.