-
Notifications
You must be signed in to change notification settings - Fork 127
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
v1.0 – Cleanup, rewrite and long-awaited customizability #76
Conversation
Because renderprops and PureComponents aren't that close friends and I don't want to have weird side effects. See https://reactjs.org/docs/render-props.html#be-careful-when-using-render-props-with-reactpurecomponent
can we merge this? works perfect and will fix this weird bug #75 |
@santomegonzalo Nice to hear that you used the new version and it worked. I'll write more usage docs later this day and then merge it. I don't want to merge it without at least a bit more documentation. |
@timomeh any news about this PR? because it's blocking a lot. |
I'm currently quite busy, in the meantime you should be able to use this PR in your package.json dependencies with |
Yeah, some people browse the docs in the downloaded package.
Summary
Cleanup of the old codebase.
Because the old codebase is really messy (what was I thinking?), this is basically is a rewrite of the whole Bottom Navigation. It will also include a large amount of customizability.
Version 1.0 will have breaking changes. It won't immediately be released with a Component for react-navigation. However, react-navigation support will be the next step.
Tasks
General
Background Ripple AnimationBottomNavigationTab ListTabControlled ComponentPress Ripple AnimationFlow Type DefinitionsTypeScript Type DefinitionsTest everythingBottomNavigationTabListPress FeedbacksTabsBadgeAdd CIUpdate READMESupport react-native-webCreate DocsPluggable Components
Icon TabIcon+Label TabShifting TabBadge