Skip to content

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

Closed
wants to merge 21 commits into from

Conversation

47PADO47
Copy link
Contributor

@47PADO47 47PADO47 commented Jul 12, 2025

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

Simulator Screenshot - iPhone 16 Pro - 2025-07-12 at 18 53 46

simulator_screenshot_C37172EB-9DD1-4D0E-84E4-F7CA1D94BDD1

@47PADO47 47PADO47 changed the title Add tab roles/search role feat: add tab roles/search role Jul 12, 2025
@okwasniewski
Copy link
Collaborator

@47PADO47 Thanks for working on this! Can you rebase with latest changes so only the changes adding tab roles are in this PR?

@47PADO47 47PADO47 force-pushed the add-tab-search-role branch from c8094e6 to 44dacd8 Compare July 15, 2025 12:33
Copy link

changeset-bot bot commented Jul 15, 2025

🦋 Changeset detected

Latest commit: 3fcb1e5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
react-native-bottom-tabs Minor
@bottom-tabs/react-navigation Minor
@bottom-tabs/expo-template Patch

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

@47PADO47 47PADO47 marked this pull request as ready for review July 15, 2025 12:40
@47PADO47
Copy link
Contributor Author

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?

Copy link
Collaborator

@okwasniewski okwasniewski left a 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?

@47PADO47
Copy link
Contributor Author

47PADO47 commented Jul 16, 2025

@okwasniewski i think everything's done, what do you say?
also i have no idea why ci keeps failing, the error message is vague and i can easily build the project on my machine without issues

@47PADO47 47PADO47 force-pushed the add-tab-search-role branch from 1f509ca to 3fcb1e5 Compare July 18, 2025 12:14
@okwasniewski okwasniewski mentioned this pull request Jul 19, 2025
@okwasniewski
Copy link
Collaborator

Thank you for working on this! Merged in #390

@47PADO47 47PADO47 deleted the add-tab-search-role branch July 19, 2025 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants