New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add compatibility with Next
#2852
base: main
Are you sure you want to change the base?
Conversation
@@ -0,0 +1,12 @@ | |||
import { findDOMNode } from 'react-dom'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe let's add a comment why this file even exists
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in f94639c.
@@ -52,7 +52,7 @@ export default class PanGestureHandler extends GestureHandler { | |||
private lastY = 0; | |||
|
|||
private activateAfterLongPress = 0; | |||
private activationTimeout = 0; | |||
private activationTimeout: ReturnType<typeof setTimeout> | undefined; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this change in a scope of this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At the beginning it was, because I added node
types into tsconfig
. Later on I've removed them, as @j-piasecki suggested (now we have @ts-ignore
, if you see better option - I'd be glad to hear it).
I brought back old types in a2185c6.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope you all know what you're doing
Description
This PR aims to bring compatibility with
Next.js
appsTest plan
Run Next.js app with Gesture Handler installed from package created from this PR.