WordPress.org

Make WordPress Core

Ticket #33029: 33029.2.diff

File 33029.2.diff, 1.1 KB (added by McGuive7, 5 years ago)

Update to simplify (switch from extra mouseup event callback to using click event)

  • wp-admin/js/common.js

     
    385385                var $this = $( this ),
    386386                        $button = $( '<button type="button" class="notice-dismiss"><span class="screen-reader-text"></span></button>' ),
    387387                        btnText = commonL10n.dismiss || '';
     388                        focusedElementAtt = 'data-wp-focused-element';
    388389
    389390                // Ensure plain text
    390391                $button.find( '.screen-reader-text' ).text( btnText );
     
    391392
    392393                $this.append( $button );
    393394
     395                // Note currently focused element before removing focus for click event.
     396                $button.on( 'mousedown.wp-dismiss-notice', function( event ) {
     397                        $( ':focus' ).attr( focusedElementAtt, '' );
     398                });
     399
    394400                $button.on( 'click.wp-dismiss-notice', function( event ) {
    395401                        event.preventDefault();
    396402                        $this.fadeTo( 100 , 0, function() {
     
    398404                                        $(this).remove();
    399405                                });
    400406                        });
     407
     408                        // Restore focus to prevously focused element.
     409                        $( '[' + focusedElementAtt + ']' ).focus().removeAttr( focusedElementAtt );
    401410                });
     411
     412
    402413        });
    403414
    404415        // Init screen meta