Changeset 37531

Timestamp:

05/23/2016 10:29:44 AM (9 years ago)

Author:

afercia

Message:

Accessibility: the Thickbox "Close" control should always be a button.

Fix the last case where the "Close" control was still a link. All the other ones
were already changed in buttons.

Fixes #36267.

Location:

trunk/src/wp-includes/js/thickbox

Files:

• thickbox.css (modified) (1 diff)
• thickbox.js (modified) (1 diff)

trunk/src/wp-includes/js/thickbox/thickbox.css

@@ -146 +146 @@
     color: #00a0d2;
 }
+
+#TB_closeWindowButton:focus .tb-close-icon {
+    -webkit-box-shadow:
+        0 0 0 1px #5b9dd9,
+        0 0 2px 1px rgba(30, 140, 190, .8);
+    box-shadow:
+        0 0 0 1px #5b9dd9,
+        0 0 2px 1px rgba(30, 140, 190, .8);
+}

trunk/src/wp-includes/js/thickbox/thickbox.js

@@ -213 +213 @@
                     if(jQuery("#TB_window").css("visibility") != "visible"){
                         if(params['modal'] != "true"){//ajax no modal
-                        jQuery("#TB_window").append("<div id='TB_title'><div id='TB_ajaxWindowTitle'>"+caption+"</div><div id='TB_closeAjaxWindow'><a href='#' id='TB_closeWindowButton'><div class='tb-close-icon'></div></a></div></div><div id='TB_ajaxContent' style='width:"+ajaxContentW+"px;height:"+ajaxContentH+"px'></div>");
+                        jQuery("#TB_window").append("<div id='TB_title'><div id='TB_ajaxWindowTitle'>"+caption+"</div><div id='TB_closeAjaxWindow'><button type='button' id='TB_closeWindowButton'><span class='screen-reader-text'>"+thickboxL10n.close+"</span><span class='tb-close-icon'></span></button></div></div><div id='TB_ajaxContent' style='width:"+ajaxContentW+"px;height:"+ajaxContentH+"px'></div>");
                         }else{//ajax modal
                         jQuery("#TB_overlay").unbind();