-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
feat: rewrite drawer for a new RNGH API #11776
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #11776 +/- ##
==========================================
+ Coverage 77.23% 77.25% +0.01%
==========================================
Files 211 209 -2
Lines 6317 6313 -4
Branches 2495 2496 +1
==========================================
- Hits 4879 4877 -2
+ Misses 1384 1382 -2
Partials 54 54 ☔ View full report in Codecov by Sentry. |
The Expo app for the example from this branch is ready! |
✅ Deploy Preview for react-navigation-example ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for react-navigation-example ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
packages/react-native-drawer-layout/src/views/GestureHandler.tsx
Outdated
Show resolved
Hide resolved
packages/react-native-drawer-layout/src/views/GestureHandler.tsx
Outdated
Show resolved
Hide resolved
f7faa22
to
1cb70e0
Compare
The Expo app for the example from this branch is ready! |
…t-navigation into @osdnk/rngh-drawer
I found an issue. When the drawer is on the right and I close it via gesture, it doesn't close fullly. Kapture.2024-03-17.at.14.35.28.mp4 |
@satya164 I addressed the problem |
This PR moves the drawer to the new API.
Screen.Recording.2024-02-11.at.20.45.55.mov