Ticket #38091: 38091.0.diff

src/wp-admin/js/customize-controls.js

@@ -808 +808 @@
                         section.template = wp.template( 'customize-themes-details-view' );
 
                         // Bind global keyboard events.
-                        $( 'body' ).on( 'keyup', function( event ) {
+                        section.container.on( 'keydown', function( event ) {
                                 if ( ! section.overlay.find( '.theme-wrap' ).is( ':visible' ) ) {
                                         return;
                                 }
@@ -826 +826 @@
                                 // Pressing the escape key fires a theme:collapse event
                                 if ( 27 === event.keyCode ) {
                                         section.closeDetails();
+                                        event.stopPropagation(); // Prevent section from being collapsed.
                                 }
                         });
 
@@ -1890 +1891 @@
                                 picker.val( value );
                                 picker.wpColorPicker( 'color', value );
                         });
+
+                        control.container.on( 'keydown', function( event ) {
+                                var pickerContainer;
+                                if ( 27 !== event.which ) { // Esc.
+                                        return;
+                                }
+                                pickerContainer = control.container.find( '.wp-picker-container' );
+                                if ( pickerContainer.hasClass( 'wp-picker-active' ) ) {
+                                        picker.wpColorPicker( 'close' );
+                                        control.container.find( '.wp-color-result' ).focus();
+                                        event.stopPropagation(); // Prevent section from being collapsed.
+                                }
+                        } );
                 }
         });
 
@@ -4002 +4016 @@
                                 return;
                         }
 
+                        /*
+                         * Abort if the event target is not the body (the default) and not inside of #customize-controls.
+                         * This ensures that ESC meant to collapse a modal dialog or a TinyMCE toolbar won't collapse something else.
+                         */
+                        if ( ! $( event.target ).is( 'body' ) && ! $.contains( $( '#customize-controls' )[0], event.target ) ) {
+                                return;
+                        }
+
                         // Check for expanded expandable controls (e.g. widgets and nav menus items), sections, and panels.
                         api.control.each( function( control ) {
                                 if ( control.expanded && control.expanded() && _.isFunction( control.collapse ) ) {