Ticket #37013: 37013.8.diff

src/wp-admin/css/common.css

@@ -3078 +3078 @@
 /* Metabox collapse arrow indicators */
 .js .sidebar-name .sidebar-name-arrow:before,
 .js .meta-box-sortables .postbox .toggle-indicator:before,
-.bulk-action-notice .toggle-indicator:before {
-        content: "\f142";
+.bulk-action-notice .toggle-indicator:before,
+.js .accordion-section .handlediv .sidebar-name-arrow:before {
         display: inline-block;
         font: normal 20px/1 dashicons;
         speak: none;
@@ -3088 +3088 @@
         text-decoration: none !important;
 }
 
+/* Open state */
+.js .sidebar-name .sidebar-name-arrow:before,
+.js .meta-box-sortables .postbox .toggle-indicator:before,
+.bulk-action-notice .toggle-indicator:before,
+.js .accordion-section.open .handlediv .sidebar-name-arrow:before {
+        content: "\f142";
+}
+
+/* Closed state */
 .js .widgets-holder-wrap.closed .sidebar-name-arrow:before,
 .js .meta-box-sortables .postbox.closed .handlediv .toggle-indicator:before,
-.bulk-action-notice .bulk-action-errors-collapsed .toggle-indicator:before {
+.bulk-action-notice .bulk-action-errors-collapsed .toggle-indicator:before,
+.js .accordion-section .handlediv .sidebar-name-arrow:before {
         content: "\f140";
 }
 
-.js .sidebar-name .sidebar-name-arrow:before {
-        padding: 10px;
-        left: 0;
-}
-
 .js #widgets-left .sidebar-name .sidebar-name-arrow {
         display: none;
 }
@@ -3141 +3146 @@
                 0 0 2px 1px rgba(30, 140, 190, .8);
 }
 
+.js .metabox-holder .accordion-section .accordion-section-title:after {
+        content: none;
+}
+
+.js .metabox-holder .accordion-section .handlediv {
+        float: right;
+        position: relative;
+        top: 11px;
+        margin-right: 11px;
+        z-index: 2;
+}
+
+.js .metabox-holder .accordion-section .handlediv .sidebar-name-arrow:before {
+        width: 20px;
+        border-radius: 50%;
+        text-indent: -1px; /* account for the dashicon alignment */
+}
+
+.js .metabox-holder .accordion-section .handlediv:focus {
+        outline: none;
+        border: none;
+        box-shadow: none;
+}
+
+.js .metabox-holder .accordion-section .handlediv:focus .sidebar-name-arrow:before {
+        box-shadow:
+                0 0 0 1px #5b9dd9,
+                0 0 2px 1px rgba(30, 140, 190, .8);
+}
+
 /* @todo: appears to be Press This only and overridden */
 #photo-add-url-div input[type="text"] {
         width: 300px;
@@ -3238 +3273 @@
 .nav-menus-php .item-edit:before,
 .widget-top .widget-action .toggle-indicator:before,
 .control-section .accordion-section-title:after,
-.accordion-section-title:after {
+.accordion-section-title:after,
+.accordion-section-arrow:after {
+        right: 0;
         content: "\f140";
         font: normal 20px/1 dashicons;
         speak: none;
@@ -3298 +3335 @@
 .control-section.open .accordion-section-title:after,
 #customize-info.open .accordion-section-title:after,
 .nav-menus-php .menu-item-edit-active .item-edit:before,
-.widget.open .widget-top .widget-action .toggle-indicator:before {
+.widget.open .widget-top .widget-action .toggle-indicator:before,
+.open .accordion-section-arrow:after {
         content: "\f142";
 }
 

src/wp-admin/css/widgets.css

@@ -193 +193 @@
         box-sizing: border-box;
 }
 
-.sidebar-name-arrow {
-        position: absolute;
-        top: 0;
-        right: 0;
-        bottom: 0;
+.js .sidebar-name .handlediv {
+        float: right;
+    width: 32px;
+    height: 32px;
+    padding: 0;
+}
+
+.js .sidebar-name .handlediv:focus {
+        box-shadow: none;
+        outline: none;
+}
+
+.js #widgets-left .sidebar-name .handlediv:focus .sidebar-name-arrow {
+        display: block;
+}
+
+.js .sidebar-name .handlediv:focus .sidebar-name-arrow:before {
+        box-shadow:
+                0 0 0 1px #5b9dd9,
+                0 0 2px 1px rgba(30, 140, 190, .8);
+}
+
+.js .sidebar-name .sidebar-name-arrow:before {
+        margin-top: 14px;
+        width: 20px;
+        border-radius: 50%;
+        text-indent: -1px; /* account for the dashicon alignment */
+}
+
+.js .widget-liquid-left .sidebar-name .sidebar-name-arrow:before {
+        margin-top: 7px;
 }
 
 .js .sidebar-name {
@@ -258 +284 @@
         margin: 0 10px 0 0;
 }
 
-#widgets-left .sidebar-name .sidebar-name-arrow:before {
-        padding: 9px;
-}
-
 #widgets-left .widgets-holder-wrap,
 div#widgets-left .widget-holder {
         background: transparent;
@@ -351 +373 @@
         padding: 15px 7px;
 }
 
-div#widgets-right .sidebar-name .sidebar-name-arrow:before {
-        top: 2px;
-}
-
 div#widgets-right .widget-top {
         padding: 0;
 }

src/wp-admin/includes/template.php

@@ -1161 +1161 @@
                                         }
                                         ?>
                                         <li class="control-section accordion-section <?php echo $hidden_class; ?> <?php echo $open_class; ?> <?php echo esc_attr( $box['id'] ); ?>" id="<?php echo esc_attr( $box['id'] ); ?>">
-                                                <h3 class="accordion-section-title hndle" tabindex="0">
+                                                <button type="button" class="handlediv button-link" aria-expanded="true">
+                                                        <span class="screen-reader-text"><?php echo sprintf( __( 'Toggle panel: %s' ), esc_html( $box['title'] ) ); ?></span>
+                                                        <span class="sidebar-name-arrow" aria-hidden="true"></span>
+                                                </button>
+                                                <h3 class="accordion-section-title hndle">
                                                         <?php echo esc_html( $box['title'] ); ?>
-                                                        <span class="screen-reader-text"><?php _e( 'Press return or enter to open this section' ); ?></span>
                                                 </h3>
                                                 <div class="accordion-section-content <?php postbox_classes( $box['id'], $page ); ?>">
                                                         <div class="inside">

src/wp-admin/includes/widgets.php

@@ -80 +80 @@
         if ( $sidebar_name ) {
                 ?>
                 <div class="sidebar-name">
-                        <div class="sidebar-name-arrow"><br /></div>
+                        <button type="button" class="handlediv button-link" aria-expanded="true">
+                                <span class="screen-reader-text"><?php echo sprintf( __( 'Toggle panel: %s' ), $sidebar_name ); ?></span>
+                                <span class="sidebar-name-arrow" aria-hidden="true"></span>
+                        </button>
                         <h2><?php echo esc_html( $sidebar_name ); ?> <span class="spinner"></span></h2>
                 </div>
                 <?php

src/wp-admin/js/accordion.js

@@ -33 +33 @@
         $( document ).ready( function () {
 
                 // Expand/Collapse accordion sections on click.
-                $( '.accordion-container' ).on( 'click keydown', '.accordion-section-title', function( e ) {
+                $( '.accordion-container' ).on( 'click keydown', '.accordion-section-title, .handlediv', function( e ) {
                         if ( e.type === 'keydown' && 13 !== e.which ) { // "return" key
                                 return;
                         }

src/wp-admin/widgets.php

@@ -387 +387 @@
 <div id="widgets-left">
         <div id="available-widgets" class="widgets-holder-wrap">
                 <div class="sidebar-name">
-                        <div class="sidebar-name-arrow"><br /></div>
+                        <button type="button" class="handlediv button-link" aria-expanded="true">
+                                <span class="screen-reader-text">Toggle panel: <?php _e( 'Available Widgets' ); ?></span>
+                                <span class="sidebar-name-arrow" aria-hidden="true"></span>
+                        </button>
                         <h2><?php _e( 'Available Widgets' ); ?> <span id="removing-widget"><?php _ex( 'Deactivate', 'removing-widget' ); ?> <span></span></span></h2>
                 </div>
                 <div class="widget-holder">