WordPress.org

Make WordPress Core


Ignore:
Timestamp:
08/03/2016 10:23:50 PM (3 years ago)
Author:
jorbin
Message:

Updates: Improve bulk update failure notice

When doing a bulk update, if there are failures the user needs to know about that. This makes it clearer that you can click on the notification to see more details, especially for screen reader users.

Fixes #37510.
Props juhise, Ankit K Gupta, afercia, jorbin, ocean90. 

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/update.php

    r38082 r38185  
    635635    </script>
    636636    <script id="tmpl-wp-bulk-updates-admin-notice" type="text/html">
    637         <div id="{{ data.id }}" class="notice <# if ( data.errors ) { #>notice-error<# } else { #>notice-success<# } #>">
     637        <div id="{{ data.id }}" class="{{ data.className }} notice <# if ( data.errors ) { #>notice-error<# } else { #>notice-success<# } #>">
    638638            <p>
    639639                <# if ( data.successes ) { #>
     
    665665                <# } #>
    666666                <# if ( data.errors ) { #>
    667                     <# if ( 1 === data.errors ) { #>
    668                         <button class="button-link">
     667                    <button class="button-link bulk-action-errors-collapsed" aria-expanded="false">
     668                        <# if ( 1 === data.errors ) { #>
    669669                            <?php
    670                             /* translators: %s: Number of failures */
    671                             printf( __( '%s failure.' ), '{{ data.errors }}' );
     670                            /* translators: %s: Number of failed updates */
     671                            printf( __( '%s update failed.' ), '{{ data.errors }}' );
    672672                            ?>
    673                         </button>
    674                     <# } else { #>
    675                         <button class="button-link">
     673                        <# } else { #>
    676674                            <?php
    677                             /* translators: %s: Number of failures */
    678                             printf( __( '%s failures.' ), '{{ data.errors }}' );
     675                            /* translators: %s: Number of failed updates */
     676                            printf( __( '%s updates failed.' ), '{{ data.errors }}' );
    679677                            ?>
    680                         </button>
    681                     <# } #>
     678                        <# } #>
     679                        <span class="screen-reader-text"><?php _e( 'Show more details' ); ?></span>
     680                        <span class="toggle-indicator" aria-hidden="true"></span>
     681                    </button>
    682682                <# } #>
    683683            </p>
    684684            <# if ( data.errors ) { #>
    685                 <ul class="hidden">
     685                <ul class="bulk-action-errors hidden">
    686686                    <# _.each( data.errorMessages, function( errorMessage ) { #>
    687687                        <li>{{ errorMessage }}</li>
Note: See TracChangeset for help on using the changeset viewer.