WordPress.org

Make WordPress Core

Ticket #31476: 31476.3.patch

File 31476.3.patch, 10.6 KB (added by afercia, 3 years ago)
  • src/wp-admin/css/common.css

     
    827827
    828828abbr.required,
    829829.file-error,
    830 .widget-control-remove:hover,
    831830.row-actions .delete a:hover,
    832831.row-actions .trash a:hover,
    833832.row-actions .spam a:hover,
     
    838837#media-items a.delete:hover,
    839838#media-items a.delete-permanently:hover,
    840839#nav-menu-footer .menu-delete:hover {
    841         color: #f00;
     840        color: #dc3232;
    842841        text-decoration: none;
    843842        border: none;
    844843}
     
    30093008
    30103009/* @todo: can we use a common class for these? */
    30113010.nav-menus-php .item-edit:before,
    3012 .widget-top a.widget-action:after,
     3011.widget-top .widget-action:after,
    30133012.control-section .accordion-section-title:after,
    30143013.accordion-section-title:after {
    30153014        right: 0;
     
    30373036}
    30383037
    30393038.widget-action:hover,
     3039.widget-action:focus,
    30403040.handlediv:hover,
    30413041.handlediv:focus,
    30423042.item-edit:hover,
     3043.item-edit:focus,
    30433044.sidebar-name:hover .sidebar-name-arrow,
    3044 .accordion-section-title:hover:after {
     3045.sidebar-name:focus .sidebar-name-arrow,
     3046.accordion-section-title:hover:after,
     3047.accordion-section-title:focus:after {
    30453048        color: #777;
    30463049}
    30473050
    3048 .widget-top a.widget-action:after {
     3051.widget-top .widget-action:after {
    30493052        padding: 1px 2px 1px 0px;
    30503053        margin-top: 10px;
    30513054        margin-right: 10px;
     
    30533056        border-radius: 50%;
    30543057}
    30553058
    3056 .widget-top a.widget-action:focus:after {
     3059.widget-top .widget-action:focus:after {
    30573060        -webkit-box-shadow:
    30583061                0 0 0 1px #5b9dd9,
    30593062                0 0 2px 1px rgba(30,140,190,.8);
     
    30723075.control-section.open .accordion-section-title:after,
    30733076#customize-info.open .accordion-section-title:after,
    30743077.nav-menus-php .menu-item-edit-active .item-edit:before,
    3075 .widget.open .widget-top a.widget-action:after {
     3078.widget.open .widget-top .widget-action:after {
    30763079        content: "\f142";
    30773080}
    30783081
     
    33963399        /* @todo: evaluate - most of these were likely replaced by dashicons */
    33973400        .curtime #timestamp,
    33983401        #screen-meta-links a.show-settings,
    3399         .widget-top a.widget-action,
    3400         .widget-top a.widget-action:hover,
     3402        .widget-top .widget-action,
     3403        .widget-top .widget-action:hover,
    34013404        .sidebar-name-arrow,
    34023405        .sidebar-name:hover .sidebar-name-arrow,
    34033406        .meta-box-sortables .postbox:hover .handlediv,
  • src/wp-admin/css/customize-controls.css

     
    33        -webkit-text-size-adjust: 100%;
    44}
    55
    6 #customize-controls a {
     6#customize-controls a,
     7#customize-controls .widget-control-remove,
     8#customize-controls .widget-control-close {
    79        text-decoration: none;
    810}
    911
  • 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
     59.widget-control-remove,
     60.widget-control-close {
     61        color: #0073aa;
     62        text-decoration: underline;
     63}
     64
     65.widget-control-remove:hover,
     66.widget-control-remove:focus,
     67.widget-control-remove:active {
     68        color: #dc3232;
     69}
     70
     71.widget-control-close:hover,
     72.widget-control-close:active {
     73        color: #00a0d2;
     74}
     75
     76.widget-control-close:focus {
     77        color: #124964;
     78}
     79
    5780/* Widget Dragging Helpers */
    5881.widget.ui-draggable-dragging {
    5982        min-width: 100%;
  • 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

     
    907907                        var self = this, $removeBtn, replaceDeleteWithRemove;
    908908
    909909                        // Configure remove button
    910                         $removeBtn = this.container.find( 'a.widget-control-remove' );
     910                        $removeBtn = this.container.find( '.widget-control-remove' );
    911911                        $removeBtn.on( 'click', function( e ) {
    912912                                e.preventDefault();
    913913
     
    943943                        } );
    944944
    945945                        replaceDeleteWithRemove = function() {
    946                                 $removeBtn.text( l10n.removeBtnLabel ); // wp_widget_control() outputs the link as "Delete"
     946                                $removeBtn.text( l10n.removeBtnLabel ); // wp_widget_control() outputs the button as "Delete"
    947947                                $removeBtn.attr( 'title', l10n.removeBtnTooltip );
    948948                        };
    949949
     
    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