hide icon fonts from screen readers #1007

Merged
merged 2 commits into from Jun 23, 2016

Conversation

Projects
None yet
2 participants
@mfairchild365
Member

mfairchild365 commented Jun 21, 2016

This is not currently an issue because in my testing, screen readers have not tried to read these specific characters. However, nothing is preventing them from doing so in the future (however unlikely). More to the point: this brings the framework in line with what we are recommending in the main content area.

That being said, there are still some cases where icon font classes are baked into parts of the template that we can't edit without releasing a new major version. See
https://github.com/unl/wdntemplates/blob/develop/Templates/fixed.dwt#L46 and https://github.com/unl/wdntemplates/blob/develop/Templates/fixed.dwt#L62

mfairchild365 added some commits Jun 21, 2016

hide icon fonts from screen readers
This is not currently an issue because in my testing, screen readers have not tried to read these characters. However, nothing is preventing them from doing so in the future (however unlikely). More to the point: this brings the framework in line with what we are recommending in the main content area.

@kabel kabel merged commit d4aeae4 into unl:develop Jun 23, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment