Opened 8 years ago
Closed 8 years ago
#38764 closed defect (bug) (fixed)
Twenty Seventeen: update screen-reader-text to reset word-wrap
Reported by: | afercia | Owned by: | davidakennedy |
---|---|---|---|
Milestone: | 4.7 | Priority: | normal |
Severity: | normal | Version: | 4.7 |
Component: | Bundled Theme | Keywords: | has-screenshots has-patch commit |
Focuses: | accessibility | Cc: |
Description
Twenty Seventeen makes use of the CSS property word-wrap
set to break-word
on a few elements, most notably #page
. When inherited by elements visually hidden with the screen-reader-text
class, screen readers may read out words ignoring spaces between words. This issue was already addressed in core, see #31962
I'd recommend to use the same solution adopted by core, resetting the property to normal where needed and, to be safe, consider to use !important
:
word-wrap: normal !important;
After resetting to normal:
Additionally, I'm not sure there's the need for the prefixed -ms-
version, apparently it was used as alias by IE 8:
https://developer.mozilla.org/en-US/docs/Web/CSS/overflow-wrap
IE8 introduced -ms-word-wrap as a synonym for word-wrap. Don't use the -ms- prefix.
Worth noting this was done also for Twenty Sixteen, see commit on GitHub. Not having history of these kind of things on SVN is a bit unfortunate.