From 33f967c04b8dc52673e98c5ab124f7307d7b17f5 Mon Sep 17 00:00:00 2001 From: Chris Rodriguez Date: Thu, 21 Jan 2016 10:55:24 -0500 Subject: [PATCH 1/2] Adjusting .sr mixins for NVDA/JAWS on Windows --- common/static/sass/_mixins.scss | 10 +++------- lms/static/sass/base/_mixins.scss | 8 ++------ 2 files changed, 5 insertions(+), 13 deletions(-) diff --git a/common/static/sass/_mixins.scss b/common/static/sass/_mixins.scss index e326dec790..57b2accbfb 100644 --- a/common/static/sass/_mixins.scss +++ b/common/static/sass/_mixins.scss @@ -404,14 +404,10 @@ // +Content - Screenreader Text - Extend // ==================== %cont-text-sr { - border: 0; - clip: rect(0 0 0 0); - height: 1px; - margin: -1px; - overflow: hidden; - padding: 0; + // clip has been deprecated but is still supported + clip: rect(1px 1px 1px 1px); + clip: rect(1px, 1px, 1px, 1px); position: absolute; - width: 1px; } // +Content - Text Wrap - Extend diff --git a/lms/static/sass/base/_mixins.scss b/lms/static/sass/base/_mixins.scss index 5443047a99..fbe41c97ac 100644 --- a/lms/static/sass/base/_mixins.scss +++ b/lms/static/sass/base/_mixins.scss @@ -125,14 +125,10 @@ // extends -hidden elems - screenreaders %text-sr { - border: 0; + // clip has been deprecated but is still supported clip: rect(1px 1px 1px 1px); - height: 1px; - margin: -1px; - overflow: hidden; - padding: 0; + clip: rect(1px, 1px, 1px, 1px); position: absolute; - width: 1px; } // extends - ensures proper contrast for automated checkers From a0a9617a86d4afc9954bc5dcce97ad91f68bbc05 Mon Sep 17 00:00:00 2001 From: Chris Rodriguez Date: Thu, 21 Jan 2016 15:18:05 -0500 Subject: [PATCH 2/2] Replacing height and width in lieu of word-wrap --- common/static/sass/_mixins.scss | 16 ++++++++++++---- lms/static/sass/base/_mixins.scss | 16 ++++++++++++---- 2 files changed, 24 insertions(+), 8 deletions(-) diff --git a/common/static/sass/_mixins.scss b/common/static/sass/_mixins.scss index 57b2accbfb..09f2e1b982 100644 --- a/common/static/sass/_mixins.scss +++ b/common/static/sass/_mixins.scss @@ -404,10 +404,18 @@ // +Content - Screenreader Text - Extend // ==================== %cont-text-sr { - // clip has been deprecated but is still supported - clip: rect(1px 1px 1px 1px); - clip: rect(1px, 1px, 1px, 1px); - position: absolute; + // clip has been deprecated but is still supported + clip: rect(1px 1px 1px 1px); + clip: rect(1px, 1px, 1px, 1px); + position: absolute; + margin: -1px; + height: 1px; + width: 1px; + border: 0; + padding: 0; + overflow: hidden; + // ensure there are spaces in sr text + word-wrap: normal; } // +Content - Text Wrap - Extend diff --git a/lms/static/sass/base/_mixins.scss b/lms/static/sass/base/_mixins.scss index fbe41c97ac..05521a2124 100644 --- a/lms/static/sass/base/_mixins.scss +++ b/lms/static/sass/base/_mixins.scss @@ -125,10 +125,18 @@ // extends -hidden elems - screenreaders %text-sr { - // clip has been deprecated but is still supported - clip: rect(1px 1px 1px 1px); - clip: rect(1px, 1px, 1px, 1px); - position: absolute; + // clip has been deprecated but is still supported + clip: rect(1px 1px 1px 1px); + clip: rect(1px, 1px, 1px, 1px); + position: absolute; + margin: -1px; + height: 1px; + width: 1px; + border: 0; + padding: 0; + overflow: hidden; + // ensure there are spaces in sr text + word-wrap: normal; } // extends - ensures proper contrast for automated checkers