WordPress.org

Make WordPress Core

Changeset 37531


Ignore:
Timestamp:
05/23/2016 10:29:44 AM (3 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:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/js/thickbox/thickbox.css

    r36964 r37531  
    146146    color: #00a0d2;
    147147}
     148
     149#TB_closeWindowButton:focus .tb-close-icon {
     150    -webkit-box-shadow:
     151        0 0 0 1px #5b9dd9,
     152        0 0 2px 1px rgba(30, 140, 190, .8);
     153    box-shadow:
     154        0 0 0 1px #5b9dd9,
     155        0 0 2px 1px rgba(30, 140, 190, .8);
     156}
  • trunk/src/wp-includes/js/thickbox/thickbox.js

    r36964 r37531  
    213213                    if(jQuery("#TB_window").css("visibility") != "visible"){
    214214                        if(params['modal'] != "true"){//ajax no modal
    215                         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>");
     215                        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>");
    216216                        }else{//ajax modal
    217217                        jQuery("#TB_overlay").unbind();
Note: See TracChangeset for help on using the changeset viewer.