From 87ea169ae5b52e54602d0b0c65f08cd8eeb2fd9d Mon Sep 17 00:00:00 2001 From: Mark Sadecki Date: Thu, 23 Jun 2016 17:40:06 -0400 Subject: [PATCH 1/2] removes text value from span that is intended only to receive focus --- lms/static/js/edxnotes/plugins/accessibility.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lms/static/js/edxnotes/plugins/accessibility.js b/lms/static/js/edxnotes/plugins/accessibility.js index 79972261d8..b8fc52dba7 100644 --- a/lms/static/js/edxnotes/plugins/accessibility.js +++ b/lms/static/js/edxnotes/plugins/accessibility.js @@ -48,8 +48,7 @@ define(['jquery', 'underscore', 'annotator_1.2.9'], function ($, _, Annotator) { addFocusGrabber: function () { this.focusGrabber = $('', { 'class': 'sr edx-notes-focus-grabber', - 'tabindex': '-1', - 'text': gettext('Focus grabber') + 'tabindex': '-1' }); this.annotator.wrapper.before(this.focusGrabber); }, From 6f9ccd8d71ada98b9b86f690699a9c03f6e99fd0 Mon Sep 17 00:00:00 2001 From: Mark Sadecki Date: Fri, 24 Jun 2016 10:18:41 -0400 Subject: [PATCH 2/2] decided to remove the sr class since this span element now has no text content --- lms/static/js/edxnotes/plugins/accessibility.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lms/static/js/edxnotes/plugins/accessibility.js b/lms/static/js/edxnotes/plugins/accessibility.js index b8fc52dba7..f9777b328f 100644 --- a/lms/static/js/edxnotes/plugins/accessibility.js +++ b/lms/static/js/edxnotes/plugins/accessibility.js @@ -47,7 +47,7 @@ define(['jquery', 'underscore', 'annotator_1.2.9'], function ($, _, Annotator) { addFocusGrabber: function () { this.focusGrabber = $('', { - 'class': 'sr edx-notes-focus-grabber', + 'class': 'edx-notes-focus-grabber', 'tabindex': '-1' }); this.annotator.wrapper.before(this.focusGrabber);