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

[Accessibility] Change the text size from dp to sp #98

Open
mohammedhemaid opened this issue Jan 17, 2020 · 2 comments
Open

[Accessibility] Change the text size from dp to sp #98

mohammedhemaid opened this issue Jan 17, 2020 · 2 comments
Assignees

Comments

@mohammedhemaid
Copy link
Collaborator

mohammedhemaid commented Jan 17, 2020

@Gear61 I saw this in the "name_choosing_settings" layout and I will check the other layouts to edit it.

@mohammedhemaid mohammedhemaid added enhancement easy Takes a few hours to do (1 - 25 lines of code) labels Jan 17, 2020
@mohammedhemaid mohammedhemaid self-assigned this Jan 17, 2020
@Gear61 Gear61 changed the title change the text size from dp tp sp Change the text size from dp to sp Jan 18, 2020
@Gear61
Copy link
Owner

Gear61 commented Jan 18, 2020

Hey @mohammedhemaid, thanks for filing this issue! There is a reason why I use dp instead of sp; sp is actually a dynamic setting that adjusts with the OS level text size setting. I feel like a lot of people don't know that (even people who work at top tech companies) and just blindly follow what the IDE tells them; using sp is actually a significant undertaking that involves auditing across all text size settings. To make things even more complicated, we use a TextView variant for icons, so it would be weird if all the icons started changing based on text size. With all that being said, are you sure you want to work on this? We would need to go through surface by surface and make sure all the text size settings worked.

@mohammedhemaid
Copy link
Collaborator Author

mohammedhemaid commented Jan 18, 2020

@Gear61 Actually I didn't know about that, I can change all the surfaces but I will keep it like this. and I will change the group dialog settings to meet the flow. Thanks for the feedback!

@Gear61 Gear61 removed the easy Takes a few hours to do (1 - 25 lines of code) label Jan 19, 2020
@Gear61 Gear61 changed the title Change the text size from dp to sp [Accessibility] Change the text size from dp to sp Jan 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants