WordPress.org

Make WordPress Core

Ticket #31233: 31233.18.diff

File 31233.18.diff, 5.2 KB (added by valendesigns, 5 years ago)

Forgot type="button"

  • src/wp-admin/css/common.css

    diff --git src/wp-admin/css/common.css src/wp-admin/css/common.css
    index 6a1e9d7..925fae2 100644
    div.error p, 
    12881288        padding: 2px;
    12891289}
    12901290
     1291.notice-dismissible {
     1292        padding-right: 38px;
     1293        position: relative;
     1294}
     1295
     1296.notice-close {
     1297        position: absolute;
     1298        top: 0;
     1299        right: 0;
     1300        border: none;
     1301        margin: 0;
     1302        padding: 0.692em;
     1303        background: none;
     1304        color: #bbb;
     1305        cursor: pointer;
     1306}
     1307
     1308@-moz-document url-prefix() {
     1309        .notice-close {
     1310                top: 1px;
     1311                right: 1px;
     1312        }
     1313}
     1314
     1315noindex:-o-prefocus, .notice-close {
     1316        height: 38px;
     1317        width: 38px;
     1318}
     1319
     1320.notice-close:hover {
     1321        color: #c00;
     1322}
     1323
     1324.notice-close:active,
     1325.notice-close:focus {
     1326        color: #c00;
     1327        outline: none;
     1328        -webkit-box-shadow: 0 0 0 1px #5b9dd9, 0 0 2px 1px rgba(30, 140, 190, .8);
     1329        box-shadow: 0 0 0 1px #5b9dd9, 0 0 2px 1px rgba(30, 140, 190, .8);
     1330}
     1331
     1332.ie8 .notice-close:focus {
     1333        outline: 1px solid #5b9dd9;
     1334}
     1335
     1336.notice-close .dashicons-dismiss {
     1337        position: relative;
     1338        top: 0;
     1339        left: 0;
     1340        vertical-align: text-bottom;
     1341}
     1342
     1343.notice-close .dashicons-dismiss:before {
     1344        vertical-align: top;
     1345}
     1346
     1347.ie8 .notice-close .dashicons-dismiss:before {
     1348        display: block;
     1349}
     1350
    12911351.notice-success,
    12921352div.updated {
    1293      border-color: #7ad03a;
     1353        border-color: #7ad03a;
    12941354}
    12951355
    12961356.notice-warning {
    1297     border-color: #ffba00;
     1357        border-color: #ffba00;
    12981358}
    12991359
    13001360.notice-error,
    13011361div.error {
    1302     border-color: #dd3d36;
     1362        border-color: #dd3d36;
    13031363}
    13041364
    13051365.notice-info {
    1306     border-color: #00a0d2;
     1366        border-color: #00a0d2;
    13071367}
    13081368
    13091369.wrap .notice,
    img { 
    31223182        }
    31233183
    31243184        /* Feedback Messages */
    3125         .wrap div.updated, .wrap div.error, .media-upload-form div.error {
     3185        .notice, .wrap div.updated, .wrap div.error, .media-upload-form div.error {
    31263186                margin: 20px 0 10px 0;
    31273187                padding: 5px 10px;
    31283188                font-size: 14px;
    31293189                line-height: 175%;
    31303190        }
    31313191
     3192        .notice-dismissible {
     3193                padding-right: 46px;
     3194        }
     3195
     3196        .notice-close {
     3197                padding: 0.929em;
     3198        }
     3199       
     3200        noindex:-o-prefocus, .notice-close {
     3201                height: 46px;
     3202                width: 46px;
     3203        }
     3204       
     3205        noindex:-o-prefocus, .notice-close .dashicons-dismiss {
     3206                vertical-align: bottom;
     3207        }
     3208
    31323209        .wrap .icon32 + h2 {
    31333210                margin-top: -2px;
    31343211        }
  • src/wp-admin/edit-form-advanced.php

    diff --git src/wp-admin/edit-form-advanced.php src/wp-admin/edit-form-advanced.php
    index 84cb32c..87cb06b 100644
    if ( isset( $post_new_file ) && current_user_can( $post_type_object->cap->create 
    410410<div id="notice" class="notice notice-warning"><p id="has-newer-autosave"><?php echo $notice ?></p></div>
    411411<?php endif; ?>
    412412<?php if ( $message ) : ?>
    413 <div id="message" class="updated"><p><?php echo $message; ?></p></div>
     413<div id="message" class="notice notice-success notice-dismissible">
     414        <p><?php echo $message; ?></p>
     415        <button type="button" class="notice-close hide-if-no-js">
     416                <span class="dashicons dashicons-dismiss">
     417                        <span class="screen-reader-text"></span>
     418                </span>
     419        </button>
     420</div>
    414421<?php endif; ?>
    415422<div id="lost-connection-notice" class="error hidden">
    416423        <p><span class="spinner"></span> <?php _e( '<strong>Connection lost.</strong> Saving has been disabled until you&#8217;re reconnected.' ); ?>
  • src/wp-admin/js/common.js

    diff --git src/wp-admin/js/common.js src/wp-admin/js/common.js
    index f879143..b13ec45 100644
    $(document).ready( function() { 
    373373        $firstHeading.nextAll( 'div.updated, div.error, div.notice' ).addClass( 'below-h2' );
    374374        $( 'div.updated, div.error, div.notice' ).not( '.below-h2, .inline' ).insertAfter( $firstHeading );
    375375
     376        // Make notices dismissible
     377        $( '.notice-dismissible' ).each( function() {
     378                var $this = $( this ),
     379                        $button = $( '.notice-close', $this ),
     380                        btnText = commonL10n.dismiss || '',
     381                        append  = false;
     382
     383                // Create the button markup if it has not been added
     384                if ( $button.length === 0 ) {
     385                        $button = $( '<button type="button" class="notice-close hide-if-no-js"><span class="dashicons dashicons-dismiss"><span class="screen-reader-text"></span></span></button>' );
     386                        append = true;
     387                }
     388
     389                // Ensure plain text
     390                $button.find( '.screen-reader-text' ).text( btnText );
     391
     392                // Only append if we need to
     393                if ( append ) {
     394                        $this.append( $button );
     395                }
     396
     397                $button.on( 'click.wp-dismiss-notice', function( event ) {
     398                        event.preventDefault();
     399                        $this.slideUp( 'fast' );
     400                });
     401        });
     402
    376403        // Init screen meta
    377404        screenMeta.init();
    378405
  • src/wp-includes/script-loader.php

    diff --git src/wp-includes/script-loader.php src/wp-includes/script-loader.php
    index 90ef799..518b3c8 100644
    function wp_default_scripts( &$scripts ) { 
    7979
    8080        $scripts->add( 'common', "/wp-admin/js/common$suffix.js", array('jquery', 'hoverIntent', 'utils'), false, 1 );
    8181        did_action( 'init' ) && $scripts->localize( 'common', 'commonL10n', array(
    82                 'warnDelete' => __("You are about to permanently delete the selected items.\n  'Cancel' to stop, 'OK' to delete.")
     82                'warnDelete' => __( "You are about to permanently delete the selected items.\n  'Cancel' to stop, 'OK' to delete." ),
     83                'dismiss'    => __( 'Dismiss this notice.' ),
    8384        ) );
    8485
    8586        $scripts->add( 'wp-a11y', "/wp-includes/js/wp-a11y$suffix.js", array( 'jquery' ), false, 1 );