-
-
Notifications
You must be signed in to change notification settings - Fork 514
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: Adds search bar on Android #1166
Conversation
252f93d
to
872aa60
Compare
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 added some observations about the code. Please answer them and keep up with the good work!
android/src/main/java/com/swmansion/rnscreens/ScreenStackFragment.kt
Outdated
Show resolved
Hide resolved
android/src/main/java/com/swmansion/rnscreens/ScreenStackHeaderSubview.kt
Show resolved
Hide resolved
android/src/main/java/com/swmansion/rnscreens/SearchBarManager.kt
Outdated
Show resolved
Hide resolved
@kacperkapusciak I updated the docs. I think I did it in all necessary places |
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.
Please resolve conflicts and fix an issue with disableBackButtonOverride
prop. Other than that LGTM! 🙌 🚀
Also, please wait for @WoLewicki approval before merging
Cheers
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.
Good job! I added some comments, please answer them and ping me after that.
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.
Thanks for answering the previous comments! I left some more and I think it is now OK to add v4 version too.
Co-authored-by: Wojciech Lewicki <wojciech.lewicki@swmansion.com>
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.
Good job! Please wait for @kacperkapusciak and we can merge it!
Description
Adds experimental version of search bar on Android
Checklist