-
-
Notifications
You must be signed in to change notification settings - Fork 191
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
Allow web's SafeAreaView to forward ref to its underlying view #383
Allow web's SafeAreaView to forward ref to its underlying view #383
Conversation
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.
Looks good, can you just fix the formatting in SafeAreaView.web.tsx?
1930497
to
10a0c32
Compare
It should be good now 🙂 |
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.
Looks good! Just a few things that might make the formatting less indented
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.
Looks good! Thanks for fixing this
My pleasure 🙂 |
@janicduplessis Thank you fast release! Am I missing something or they are just lost on their way to production? |
I might have messed something up, I published 4.5.3 which seems fine. Thanks for finding this. |
Summary
This PR solves the issue #295 by adding the ability for the web's
SafeAreaView
to forward the ref to the underlyingView
.Test Plan
I run the code by creating a basic Expo app and patching it with my changes.