Changeset 55326

Timestamp:

02/14/2023 12:36:32 AM (4 years ago)

Author:

joedolson

Message:

Administration: Close admin menu when focus moves to body.

User should not have to reach the admin menu toggle in order to close the menu. This can be a problem for one-handed mobile use, users with small hands, and numerous other situational usages.

Close the admin menu when focus moves anywhere other than the menu or the menu toggle and the current document is active.

Props kaneva, sabernhardt, costdev, ryokuhi, hellofromtonya, dhusakovic, thelovekesh, joedolson.
Fixes #53587.

File:

• trunk/src/js/_enqueues/admin/common.js (modified) (2 diffs)

trunk/src/js/_enqueues/admin/common.js

@@ -1703 +1703 @@
                         } );
 
+                        // Close sidebar when focus moves outside of toggle and sidebar.
+                        $( '#wp-admin-bar-menu-toggle, #adminmenumain' ).on( 'focusout', function( e ) {
+                                var focusIsInToggle, focusIsInSidebar;
+
+                                if ( ! $wpwrap.hasClass( 'wp-responsive-open' ) || ! document.hasFocus() ) {
+                                        return;
+                                }
+                                // A brief delay is required to allow focus to switch to another element.
+                                setTimeout( function() {
+                                        focusIsInToggle  = $.contains( $( '#wp-admin-bar-menu-toggle' )[0], $( ':focus' )[0] );
+                                        focusIsInSidebar = $.contains( $( '#adminmenumain' )[0], $( ':focus' )[0] );
+
+                                        if ( ! focusIsInToggle && ! focusIsInSidebar ) {
+                                                $( '#wp-admin-bar-menu-toggle' ).trigger( 'click.wp-responsive' );
+                                        }
+                                }, 10 );
+                        } );
+
+
                         // Add menu events.
                         $adminmenu.on( 'click.wp-responsive', 'li.wp-has-submenu > a', function( event ) {
@@ -1710 +1729 @@
 
                                 $( this ).parent( 'li' ).toggleClass( 'selected' );
+                                $( this ).trigger( 'focus' );
                                 event.preventDefault();
                         });