-
-
Notifications
You must be signed in to change notification settings - Fork 576
feat(iOS, Tabs): add support for tabBarMinimizeBehavior #3011
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
Conversation
As for our discussion about possibility of using a macro, I tested it and it works. I created a ticket for it (https://github.com/software-mansion/react-native-screens-labs/issues/286) because it would probably be cleaner to do it in separate PR and refactor all usages of those checks. |
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.
overall looks good 👍
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!
I've created https://github.com/software-mansion/react-native-screens-labs/issues/305 with an issue report that concerns badge behaviour, but I don't think it is in scope of this ticket. Most likely this is a native bug, however we need to verify that.
320381d
to
02813e7
Compare
cf6ee6a
to
6b923a9
Compare
Description
Adds support for
tabBarMinimizeBehavior
.Closes https://github.com/software-mansion/react-native-screens-labs/issues/178.
Moved from https://github.com/software-mansion/react-native-screens-labs/pull/214.
Changes
tabBarMinimizeBehavior
Test code and steps to reproduce
Add
tabBarMinimizeBehavior
forBottomTabs
component, e.g.onScrollDown
and check behavior with ScrollView.Checklist