Skip to content
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

Bugs + (partial) Fixes: Non-responsiveness of "tabBarLabelStyle" and "tabBarLabel" in MaterialTopTabBar #11968

Closed
wants to merge 5 commits into from

Conversation

DrStoop
Copy link

@DrStoop DrStoop commented Apr 28, 2024

The props tabBarLabelStyle and tabBarLabel are ignored by MaterialTopTabBar. This pull requests solves the problem full for tabBarLabelStyle. For tabBarLabel this PR only fixes the problem for tabBarLabel(s) of type string.

MaterialTopTabBar is ignoring the prop "tabBarLableStyle", so the label style couldn't be changed. The bug was that the MaterialLabelType variable "lableStyle" is incompatible with the "label" input variable of customlabel in TabBarItemInternal (react-native-tab-view).
The prop "tabBarLabel" is ignored by MaterialTopTabBar. It only displays the route name. This fix allows strings for "tabBarLabel" but no components (as suggested by docs).
Copy link

Hey @DrStoop! Thanks for opening your first pull request in this repo. If you haven't already, make sure to read our contribution guidelines.

Copy link

Hey @autofix-ci[bot]! Thanks for opening your first pull request in this repo. If you haven't already, make sure to read our contribution guidelines.

Copy link

netlify bot commented Apr 28, 2024

Deploy Preview for react-navigation-example ready!

Name Link
🔨 Latest commit 3bb6a2e
🔍 Latest deploy log https://app.netlify.com/sites/react-navigation-example/deploys/662e326b2ee83a0008a4011e
😎 Deploy Preview https://deploy-preview-11968--react-navigation-example.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Apr 28, 2024

Deploy Preview for react-navigation-example ready!

Name Link
🔨 Latest commit 726c6fc
🔍 Latest deploy log https://app.netlify.com/sites/react-navigation-example/deploys/663cef4a87db3e00086f50b6
😎 Deploy Preview https://deploy-preview-11968--react-navigation-example.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

labelStyle is not being passed through to TabBarItem -> TabBarItemInternal
@DrStoop DrStoop closed this May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant