WordPress.org
Get WordPress

Make WordPress Core

Ticket #31476: 31476.2.patch

File 31476.2.patch, 8.7 KB (added by afercia, 9 years ago)

  • src/wp-admin/css/common.css

     
    29552955
    29562956/* @todo: can we use a common class for these? */
    29572957.nav-menus-php .item-edit:before,
    2958 .widget-top a.widget-action:after,
     2958.widget-top .widget-action:after,
    29592959.control-section .accordion-section-title:after,
    29602960.accordion-section-title:after {
    29612961        right: 0;
    … …  
    29832983}
    29842984
    29852985.widget-action:hover,
     2986.widget-action:focus,
    29862987.handlediv:hover,
    29872988.handlediv:focus,
    29882989.item-edit:hover,
     2990.item-edit:focus,
    29892991.sidebar-name:hover .sidebar-name-arrow,
    2990 .accordion-section-title:hover:after {
     2992.sidebar-name:focus .sidebar-name-arrow,
     2993.accordion-section-title:hover:after,
     2994.accordion-section-title:focus:after {
    29912995        color: #777;
    29922996}
    29932997
    2994 .widget-top a.widget-action:after {
     2998.widget-top .widget-action:after {
    29952999        padding: 1px 2px 1px 0px;
    29963000        margin-top: 10px;
    29973001        margin-right: 10px;
    … …  
    29993003        border-radius: 50%;
    30003004}
    30013005
    3002 .widget-top a.widget-action:focus:after {
     3006.widget-top .widget-action:focus:after {
    30033007        -webkit-box-shadow:
    30043008                0 0 0 1px #5b9dd9,
    30053009                0 0 2px 1px rgba(30,140,190,.8);
    … …  
    30183022.control-section.open .accordion-section-title:after,
    30193023#customize-info.open .accordion-section-title:after,
    30203024.nav-menus-php .menu-item-edit-active .item-edit:before,
    3021 .widget.open .widget-top a.widget-action:after {
     3025.widget.open .widget-top .widget-action:after {
    30223026        content: "\f142";
    30233027}
    30243028
    … …  
    33423346        /* @todo: evaluate - most of these were likely replaced by dashicons */
    33433347        .curtime #timestamp,
    33443348        #screen-meta-links a.show-settings,
    3345         .widget-top a.widget-action,
    3346         .widget-top a.widget-action:hover,
     3349        .widget-top .widget-action,
     3350        .widget-top .widget-action:hover,
    33473351        .sidebar-name-arrow,
    33483352        .sidebar-name:hover .sidebar-name-arrow,
    33493353        .meta-box-sortables .postbox:hover .handlediv,

  • src/wp-admin/css/customize-widgets.css

     
    8383        cursor: move;
    8484}
    8585
    86 .customize-control-widget_form.expanded a.widget-action:after {
     86.customize-control-widget_form.expanded .widget-action:after {
    8787        content: "\f142";
    8888}
    8989
    90 .customize-control-widget_form.wide-widget-control a.widget-action:after {
     90.customize-control-widget_form.wide-widget-control .widget-action:after {
    9191        content: "\f139";
    9292}
    9393
    94 .customize-control-widget_form.wide-widget-control.expanded a.widget-action:after {
     94.customize-control-widget_form.wide-widget-control.expanded .widget-action:after {
    9595        content: "\f141";
    9696}
    9797

  • src/wp-admin/css/widgets.css

     
    1414        background: #f7f7f7;
    1515}
    1616
    17 .widget-top a.widget-action,
    18 .widget-top a.widget-action:hover {
     17.widget-top .widget-action,
     18.widget-top .widget-action:hover,
     19.widget-top .button-link.widget-action:focus {
     20        padding: 0;
    1921        -webkit-box-shadow: none;
    2022        box-shadow: none;
    2123        outline: none;
    … …  
    5052}
    5153
    5254.deleting .widget-title,
    53 .deleting .widget-top a.widget-action:after {
     55.deleting .widget-top .widget-action:after {
    5456        color: #a0a5aa;
    5557}
    5658

  • src/wp-admin/includes/widgets.php

     
    219219        echo $sidebar_args['before_widget']; ?>
    220220        <div class="widget-top">
    221221        <div class="widget-title-action">
    222                 <a class="widget-action hide-if-no-js" href="#available-widgets"></a>
     222                <button type="button" class="button-link widget-action hide-if-no-js" aria-expanded="false">
     223                        <span class="screen-reader-text"><?php printf( __( 'Edit widget: %s' ), $widget_title ); ?></span>
     224                </button>
    223225                <a class="widget-control-edit hide-if-js" href="<?php echo esc_url( add_query_arg( $query_arg ) ); ?>">
    224226                        <span class="edit"><?php _ex( 'Edit', 'widget' ); ?></span>
    225227                        <span class="add"><?php _ex( 'Add', 'widget' ); ?></span>
    … …  
    250252
    251253        <div class="widget-control-actions">
    252254                <div class="alignleft">
    253                 <a class="widget-control-remove" href="#remove"><?php _e('Delete'); ?></a> |
    254                 <a class="widget-control-close" href="#close"><?php _e('Close'); ?></a>
     255                <button type="button" class="button-link widget-control-remove"><?php _e( 'Delete' ); ?></button> |
     256                <button type="button" class="button-link widget-control-close"><?php _e( 'Close' ); ?></button>
    255257                </div>
    256258                <div class="alignright<?php if ( 'noform' === $has_form ) echo ' widget-control-noform'; ?>">
    257259                        <?php submit_button( __( 'Save' ), 'button-primary widget-control-save right', 'savewidget', false, array( 'id' => 'widget-' . esc_attr( $id_format ) . '-savewidget' ) ); ?>

  • src/wp-admin/js/customize-widgets.js

     
    13221322                 * @param {Object} args  merged on top of this.defaultActiveArguments
    13231323                 */
    13241324                onChangeExpanded: function ( expanded, args ) {
    1325                         var self = this, $widget, $inside, complete, prevComplete;
     1325                        var self = this, $widget, $inside, complete, prevComplete, $toggleBtn;
    13261326
    13271327                        self.embedWidgetControl(); // Make sure the outer form is embedded so that the expanded state can be set in the UI.
    13281328                        if ( expanded ) {
    … …  
    13411341
    13421342                        $widget = this.container.find( 'div.widget:first' );
    13431343                        $inside = $widget.find( '.widget-inside:first' );
     1344                        $toggleBtn = this.container.find( '.widget-top button.widget-action' );
    13441345
    13451346                        if ( expanded ) {
    13461347
    … …  
    13581359                                complete = function() {
    13591360                                        self.container.removeClass( 'expanding' );
    13601361                                        self.container.addClass( 'expanded' );
     1362                                        $toggleBtn.attr( 'aria-expanded', 'true' );
    13611363                                        self.container.trigger( 'expanded' );
    13621364                                };
    13631365                                if ( args.completeCallback ) {
    … …  
    13811383                                complete = function() {
    13821384                                        self.container.removeClass( 'collapsing' );
    13831385                                        self.container.removeClass( 'expanded' );
     1386                                        $toggleBtn.attr( 'aria-expanded', 'false' );
    13841387                                        self.container.trigger( 'collapsed' );
    13851388                                };
    13861389                                if ( args.completeCallback ) {

  • src/wp-admin/js/widgets.js

     
    4141                $(document.body).bind('click.widgets-toggle', function(e) {
    4242                        var target = $(e.target),
    4343                                css = { 'z-index': 100 },
    44                                 widget, inside, targetWidth, widgetWidth, margin;
     44                                widget, inside, targetWidth, widgetWidth, margin,
     45                                toggleBtn = target.closest( '.widget' ).find( '.widget-top button.widget-action' );
    4546
    4647                        if ( target.parents('.widget-top').length && ! target.parents('#available-widgets').length ) {
    4748                                widget = target.closest('div.widget');
    … …  
    6162                                                widget.css( css );
    6263                                        }
    6364                                        widget.addClass( 'open' );
     65                                        toggleBtn.attr( 'aria-expanded', 'true' );
    6466                                        inside.slideDown('fast');
    6567                                } else {
    6668                                        inside.slideUp('fast', function() {
    6769                                                widget.attr( 'style', '' );
    6870                                                widget.removeClass( 'open' );
     71                                                toggleBtn.attr( 'aria-expanded', 'false' );
    6972                                        });
    7073                                }
    7174                                e.preventDefault();
    … …  
    7881                        } else if ( target.hasClass('widget-control-close') ) {
    7982                                widget = target.closest('div.widget');
    8083                                widget.removeClass( 'open' );
     84                                toggleBtn.attr( 'aria-expanded', 'false' );
    8185                                wpWidgets.close( widget );
    8286                                e.preventDefault();
    8387                        } else if ( target.attr( 'id' ) === 'inactive-widgets-control-remove' ) {
    … …  
    9296                        wpWidgets.appendTitle( this );
    9397
    9498                        if ( $this.find( 'p.widget-error' ).length ) {
    95                                 $this.find( 'a.widget-action' ).trigger('click');
     99+                               $this.find( '.widget-action' ).trigger( 'click' ).attr( 'aria-expanded', 'true' );
    96100                        }
    97101                });
    98102
    … …  
    182186
    183187                                if ( inside.css('display') === 'block' ) {
    184188                                        ui.item.removeClass('open');
     189                                        ui.item.find( '.widget-top button.widget-action' ).attr( 'aria-expanded', 'false' );
    185190                                        inside.hide();
    186191                                        $(this).sortable('refreshPositions');
    187192                                }
    … …  
    254259                                }
    255260
    256261                                if ( addNew ) {
    257                                         $widget.find( 'a.widget-action' ).trigger('click');
     262                                        $widget.find( '.widget-action' ).trigger( 'click' );
    258263                                } else {
    259264                                        wpWidgets.saveOrder( $sidebar.attr('id') );
    260265                                }
    … …  
    492497
    493498        close : function(widget) {
    494499                widget.children('.widget-inside').slideUp('fast', function() {
    495                         widget.attr( 'style', '' );
     500                        widget.attr( 'style', '' ).find( '.widget-top button.widget-action' ).attr( 'aria-expanded', 'false' );
    496501                });
    497502        },
    498503