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

Fix bottomTab truncated text on iPad #6784

Merged
merged 2 commits into from
Nov 21, 2020
Merged

Fix bottomTab truncated text on iPad #6784

merged 2 commits into from
Nov 21, 2020

Conversation

yogevbd
Copy link
Collaborator

@yogevbd yogevbd commented Nov 21, 2020

Closes #6533

@guyca
Copy link
Collaborator

guyca commented Nov 21, 2020

@yogevbd Seems like the two lines this PR deletes were added in b01629c. Are we certain this change won't cause regressions?

We gotta increase our screenshots coverage to cover BottomTabs as well. These iOS issues are madness.

@yogevbd
Copy link
Collaborator Author

yogevbd commented Nov 21, 2020

The issue described in #6005 doesn't reproduce on iOS 14.2.
In this case where it's an iOS new feature bug, I'm not sure how screenshot tests will help but let's think about it..

@guyca
Copy link
Collaborator

guyca commented Nov 21, 2020

We can use screenshot tests to ensure the tab item title isn't truncated in various font configuration.

@guyca guyca merged commit ee0aafe into master Nov 21, 2020
@guyca guyca deleted the fixInlineTabItem branch November 21, 2020 13:44
@ryanbourneuk
Copy link

Hey @guyca & @yogevbd!

I've just updated to v7.8.1, and I've noticed on iOS 13.3 (iPhone 11 Pro simulator) my tab bar text is now truncated! With the changes in this PR reverted locally, they're fixed again. The iOS 14.2 (iPhone 12 Pro simulator) works with and without the lines being re-added.

I'm happy to patch-package this locally (as my client's project does not use text for tabs on iPad), but it may be worth investigating & resolving so both iPad and iPhone can show text.

Happy to help in any way I can, let me know if you need more details.

Thanks! 😄

@filipsuk
Copy link

I also had to use patch-package to revert this change. iPhone 8 simulator iOS 14.4 (and others) had issues with truncated text in the bottom menu.

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.

Tab bar text cut off and ellipsized on iPadOS 14
4 participants