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

Icons don't resize with text #1569

Closed
hannalaakso opened this issue Sep 12, 2019 · 1 comment · Fixed by #1584
Closed

Icons don't resize with text #1569

hannalaakso opened this issue Sep 12, 2019 · 1 comment · Fixed by #1584
Assignees
Labels
🕔 hours A well understood issue which we expect to take less than a day to resolve.

Comments

@hannalaakso
Copy link
Member

hannalaakso commented Sep 12, 2019

WCAG 2.1 requires that user can resize text to at least 200% . When text is resized (see #1519), our icons don't resize with text. We might not wish them to, this issue is partly for reference.

Crown icon when text resized to 32 (16 x 2 = 200%)

(Once #1568 is fixed)
Screen Shot 2019-09-12 at 16 55 40

Warning icon when text resized to 200% in Chrome

Screen Shot 2019-09-12 at 16 52 34

Arrow icon when text resized to 200%

Screen Shot 2019-09-12 at 16 57 29

How text was resized in browsers

Chrome

Settings > Appearance > Customise fonts

Firefox

Set View > Zoom > Zoom text only. Press +/-
or Preferences > Language and appearance > Font size

Safari

To increase the font size, hold down Alt and click on 'View' menu > Make text bigger
To decrease the font size, hold down Alt and click on 'View' menu > Make text smaller

Edge

Can't resize text apart from "Reading view" which removes form elements from the view.

IE 8-11

Page > Text size > Largest

iOS

Text can't resized in browser. See #882

Android Chrome/Firefox

Settings > Accessibility > Text scaling

@NickColley
Copy link
Contributor

NickColley commented Sep 18, 2019

We're going to only fix the icons that lose their meaning when scaled up, in the examples shown here that means the warning icon but there may be other components, so we should double check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕔 hours A well understood issue which we expect to take less than a day to resolve.
Projects
Development

Successfully merging a pull request may close this issue.

3 participants