Removed aria-pressed from Share widget button, at the request of the …

…AWG, to minimze confusion from label changing as well.
wet-boew · Aug 14, 2012 · 2d065dc · 2d065dc
1 parent d874d09
commit 2d065dc
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/build/grids/css/util-ie-min.css b/build/grids/css/util-ie-min.css
diff --git a/build/grids/css/util-min.css b/build/grids/css/util-min.css
diff --git a/build/js/pe-ap-min.js b/build/js/pe-ap-min.js
diff --git a/src/js/workers/share.js b/src/js/workers/share.js
@@ -93,7 +93,7 @@
 					}
 				});
 				$popupText = elm.find('.bookmark_popup_text').off('click vclick keydown').wrap('<' + opts.popupTag + ' />');
-				$popupText.attr('role', 'button').attr('aria-controls', 'bookmark_popup').attr('aria-pressed', 'false').on("click vclick keydown", function (e) {
+				$popupText.attr('role', 'button').attr('aria-controls', 'bookmark_popup').on("click vclick keydown", function (e) {
 					if (e.type === "keydown") {
 						if (!(e.ctrlKey || e.altKey || e.metaKey)) {
 							if (e.keyCode === 13 || e.keyCode === 32) { // enter or space
@@ -216,11 +216,11 @@
 							$popup.trigger("closenofocus");
 						}
 					} else if (e.type === "open") { // Open the popup menu an put the focus on the first link
-						$popupText.attr('aria-pressed', 'true').text(opts.hideText + opts.popupText);
+						$popupText.text(opts.hideText + opts.popupText);
 						$popup.attr('aria-hidden', 'false').show();
 						pe.focus($popup.show().find('li a').first());
 					} else if (e.type === "close" || e.type === "closenofocus") { // Close the popup menu
-						$popupText.attr('aria-pressed', 'false').text(opts.popupText);
+						$popupText.text(opts.popupText);
 						$popup.attr('aria-hidden', 'true').hide();
 						if (e.type === "close") {
 							pe.focus($popupText.first());