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

Make the NUX button wordwrap so that iOS 12.4 users can read it #449

Closed
wants to merge 1 commit into from

Conversation

mindgraffiti
Copy link
Contributor

@mindgraffiti mindgraffiti commented Sep 9, 2020

Fixes #155

WPiOS testing PR: tk

There is an issue where iOS 12.0+ users on smaller devices do not get proper font scaling. This is an iOS 12 issue that doesn't happen on iOS 13. Instead of attempting to force iOS 12 to use the correct font scaling, we're going to word-wrap the long text in the button so it can be successfully read by the user.

Note that the buttons in the prologue are different because the UL&S project had updated designs to work from.

Before After

@mindgraffiti mindgraffiti added the bug Something isn't working label Sep 9, 2020
@mindgraffiti mindgraffiti self-assigned this Sep 9, 2020
@mindgraffiti mindgraffiti requested review from ScoutHarris and Gio2018 and removed request for ScoutHarris and Gio2018 September 9, 2020 21:35
@mindgraffiti
Copy link
Contributor Author

Ran out of time today. Will set up testing branch tomorrow.

Base automatically changed from release/1.24.1 to trunk September 9, 2020 21:51
@mindgraffiti
Copy link
Contributor Author

Will open a fresh PR with the correct parent branch.

@mindgraffiti mindgraffiti deleted the fix/nux-button-text branch September 10, 2020 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant