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

P2 Breathe - change default font to em or rem base instead of px for better accessibility #42

Closed
pauljacobson opened this issue Mar 19, 2018 · 1 comment
Labels
[Pri] Low [Type] Enhancement New feature or request
Milestone

Comments

@pauljacobson
Copy link

Steps to replicate

Some P2s are configured with a fixed font, such as 14px. I noticed this on some internal Automattic P2s. For example:

Result

This setting ignores a user's browser font preferences, and delivers what could be a small font for users who prefer larger text.

Expected

Changing the default font-size setting to something like em or rem should improve accessibility for users who prefer larger font sizing.

@pauljacobson pauljacobson added the [Type] Enhancement New feature or request label Mar 19, 2018
@laurelfulford laurelfulford added this to the P2 Breathe milestone Jul 4, 2018
@davidakennedy
Copy link
Contributor

This is a great suggestion. However, changing the font units on an older design active on so many sites could have unintended consequences. So I'm going to close this. We should address this type of feedback in a P2 redesign in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Pri] Low [Type] Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants