-
Notifications
You must be signed in to change notification settings - Fork 52
feat: add tab roles/search role #387
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
@47PADO47 Thanks for working on this! Can you rebase with latest changes so only the changes adding tab roles are in this PR? |
c8094e6
to
44dacd8
Compare
🦋 Changeset detectedLatest commit: 3fcb1e5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
done, let me know if anything needs to be changed. i also added a changelog entry, but noticed i forgot to add one for the pr #364. should i create it? |
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 working on this!
Can you also update documentation for both React Navigation usage and standalone usage?
packages/react-native-bottom-tabs/android/src/main/java/com/rcttabview/RCTTabViewImpl.kt
Outdated
Show resolved
Hide resolved
@okwasniewski i think everything's done, what do you say? |
fix: sync Podfile.lock
1f509ca
to
3fcb1e5
Compare
Thank you for working on this! Merged in #390 |
PR Description
This pr adds support for ios tab roles
How to test?
Run the app on a mac and go into "sf symbols" example
Screenshots