Changeset 38520

Timestamp:

09/04/2016 09:50:22 PM (10 years ago)

Author:

afercia

Message:

Accessibility: Improve the Customizer and Theme Installer initial focus.

The Customizer and Theme Installer open in full overlays that need to receive
focus. Also, keyboard navigation should be constrained within the overlays. Using
CSS visibility to hide all the content except the overlays, makes them the only
available and focusable content and allows browsers to handle focus natively.

See #29158.
Fixes #33228, #27705.

Location:

trunk/src

Files:

• wp-admin/css/themes.css (modified) (2 diffs)
• wp-admin/js/customize-controls.js (modified) (2 diffs)
• wp-admin/js/theme.js (modified) (3 diffs)
• wp-includes/js/customize-loader.js (modified) (4 diffs)

trunk/src/wp-admin/css/themes.css

@@ -1183 +1183 @@
 body.full-overlay-active {
         overflow: hidden;
+        /* Hide all the content, the Customizer overlay is then made visible to be the only available content. */
+        visibility: hidden;
 }
 
@@ -1614 +1616 @@
 }
 
-.customize-active #customize-container {
-        display: block;
+/* Make the Customizer and Theme installer overlays the only available content. */
+#customize-container,
+.theme-install-overlay {
+        visibility: visible;
 }
 

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

@@ -3621 +3621 @@
                 }
 
-                var parent, topFocus,
+                var parent,
                         body = $( document.body ),
                         overlay = body.children( '.wp-full-overlay' ),
@@ -4235 +4235 @@
 
                 api.trigger( 'ready' );
-
-                // Make sure left column gets focus
-                topFocus = closeBtn;
-                topFocus.focus();
-                setTimeout(function () {
-                        topFocus.focus();
-                }, 200);
-
         });
 

trunk/src/wp-admin/js/theme.js

@@ -867 +867 @@
 
         render: function() {
-                var self = this, currentPreviewDevice,
-                        data = this.model.toJSON();
+                var self = this,
+                        currentPreviewDevice,
+                        data = this.model.toJSON(),
+                        $body = $( document.body );
+
+                $body.attr( 'aria-busy', 'true' );
 
                 this.$el.removeClass( 'iframe-ready' ).html( this.html( data ) );
@@ -880 +884 @@
 
                 this.$el.fadeIn( 200, function() {
-                        $( 'body' ).addClass( 'theme-installer-active full-overlay-active' );
-                        $( '.close-full-overlay' ).focus();
+                        $body.addClass( 'theme-installer-active full-overlay-active' );
                 });
 
@@ -891 +894 @@
         iframeLoaded: function() {
                 this.$el.addClass( 'iframe-ready' );
+                $( document.body ).attr( 'aria-busy', 'false' );
         },
 

trunk/src/wp-includes/js/customize-loader.js

@@ -184 +184 @@
                  */
                 opened: function() {
-                        Loader.body.addClass( 'customize-active full-overlay-active' );
-                },
-
-                /**
-                 * Close the Customizer overlay and return focus to the link that opened it.
+                        Loader.body.addClass( 'customize-active full-overlay-active' ).attr( 'aria-busy', 'true' );
+                },
+
+                /**
+                 * Close the Customizer overlay.
                  */
                 close: function() {
@@ -209 +209 @@
                         if ( this.originalDocumentTitle ) {
                                 document.title = this.originalDocumentTitle;
-                        }
-
-                        // Return focus to link that was originally clicked.
-                        if ( this.link ) {
-                                this.link.focus();
                         }
                 },
@@ -228 +223 @@
                         Loader.body.removeClass( 'customize-active full-overlay-active' ).removeClass( 'customize-loading' );
                         $( window ).off( 'beforeunload', Loader.beforeunload );
+                        /*
+                         * Return focus to the link that opened the Customizer overlay after
+                         * the body element visibility is restored.
+                         */
+                        if ( Loader.link ) {
+                                Loader.link.focus();
+                        }
                 },
 
@@ -234 +236 @@
                  */
                 loaded: function() {
-                        Loader.body.removeClass('customize-loading');
+                        Loader.body.removeClass( 'customize-loading' ).attr( 'aria-busy', 'false' );
                 },