Ticket #34613: 34613.patch

src/wp-admin/includes/class-wp-terms-list-table.php

@@ -588 +588 @@
         ?>
 
                 <p class="inline-edit-save submit">
-                        <a href="#inline-edit" class="cancel button-secondary alignleft"><?php _e( 'Cancel' ); ?></a>
-                        <a href="#inline-edit" class="save button-primary alignright"><?php echo $tax->labels->update_item; ?></a>
+                        <button type="button" class="cancel button-secondary alignleft"><?php _e( 'Cancel' ); ?></button>
+                        <button type="button" class="save button-primary alignright"><?php echo $tax->labels->update_item; ?></button>
                         <span class="spinner"></span>
                         <span class="error" style="display:none;"></span>
                         <?php wp_nonce_field( 'taxinlineeditnonce', '_inline_edit', false ); ?>

src/wp-admin/js/inline-edit-tax.js

@@ -1 +1 @@
 /* global inlineEditL10n, ajaxurl */
+window.wp = window.wp || {};
 
 var inlineEditTax;
-(function($) {
+( function( $, wp ) {
 inlineEditTax = {
 
         init : function() {
@@ -22 +23 @@
                         }
                 });
 
-                $( 'a.cancel', row ).click( function() {
+                $( '.cancel', row ).click( function() {
                         return inlineEditTax.revert();
                 });
-                $( 'a.save', row ).click( function() {
+                $( '.save', row ).click( function() {
                         return inlineEditTax.save(this);
                 });
                 $( 'input, select', row ).keydown( function( e ) {
@@ -117 +118 @@
                                                 // Update the value in the Parent dropdown.
                                                 $( '#parent' ).find( 'option[value=' + option_value + ']' ).text( row.find( '.row-title' ).text() );
 
-                                                row.hide().fadeIn();
+                                                row.hide().fadeIn( 400, function() {
+                                                        // Move focus back to the taxonomy title.
+                                                        wp.a11y.speak( inlineEditL10n.saved );
+                                                        row.find( '.row-title' ).focus();
+                                                });
+
                                         } else {
                                                 $('#edit-'+id+' .inline-edit-save .error').html(r).show();
+                                                wp.a11y.speak( r );
                                         }
                                 } else {
                                         $('#edit-'+id+' .inline-edit-save .error').html(inlineEditL10n.error).show();
+                                        wp.a11y.speak( inlineEditL10n.error );
                                 }
                         }
                 );
@@ -136 +144 @@
                         $( 'table.widefat .spinner' ).removeClass( 'is-active' );
                         $('#'+id).siblings('tr.hidden').addBack().remove();
                         id = id.substr( id.lastIndexOf('-') + 1 );
-                        $(this.what+id).show();
+                        // Show the taxonomy listing and move focus back to the taxonomy title.
+                        $( this.what+id ).show().find( '.row-title' ).focus();
                 }
 
                 return false;
@@ -149 +158 @@
 };
 
 $(document).ready(function(){inlineEditTax.init();});
-})(jQuery);
+})( jQuery, window.wp);

src/wp-includes/script-loader.php

@@ -562 +562 @@
                         'comma' => trim( _x( ',', 'tag delimiter' ) ),
                 ) );
 
-                $scripts->add( 'inline-edit-tax', "/wp-admin/js/inline-edit-tax$suffix.js", array( 'jquery' ), false, 1 );
+                $scripts->add( 'inline-edit-tax', "/wp-admin/js/inline-edit-tax$suffix.js", array( 'jquery', 'wp-a11y' ), false, 1 );
                 did_action( 'init' ) && $scripts->localize( 'inline-edit-tax', 'inlineEditL10n', array(
-                        'error' => __('Error while saving the changes.')
+                        'error' => __('Error while saving the changes.'),
+                        'saved' => __( 'Changes saved' ),
                 ) );
 
                 $scripts->add( 'plugin-install', "/wp-admin/js/plugin-install$suffix.js", array( 'jquery', 'thickbox' ), false, 1 );