WordPress.org

Make WordPress Core

Ticket #29326: 29326.3.diff

File 29326.3.diff, 2.5 KB (added by adamsilverstein, 7 years ago)

further refine edit selection a11y

  • src/wp-includes/css/media-views.css

     
    923923}
    924924
    925925.attachment .close {
    926         display: none;
     926        display: block;
    927927        position: absolute;
    928928        top: 5px;
    929929        right: 5px;
     
    942942        border-radius: 3px;
    943943        -webkit-box-shadow: 0 0 0 1px rgba( 0, 0, 0, 0.3 );
    944944        box-shadow: 0 0 0 1px rgba( 0, 0, 0, 0.3 );
     945        transition-duration: none;
     946        transition-property: none;
    945947}
    946948
    947 .attachment .close:hover {
     949.attachment a.close:hover,
     950.attachment a.close:focus {
    948951        -webkit-box-shadow: 0 0 0 1px rgba( 0, 0, 0, 0.6 );
    949952        box-shadow: 0 0 0 1px rgba( 0, 0, 0, 0.6 );
    950 }
     953        background-position: -36px 4px;
    951954
    952 .attachment:hover .close {
    953         display: block;
    954955}
    955956
    956957.attachment .check {
  • src/wp-includes/js/media-views.js

     
    30193019
    30203020                        // Browse our library of attachments.
    30213021                        this.content.set( view );
     3022
     3023                        // Trigger the controller to set focus
     3024                        view.controller.trigger( 'edit:selection', this );
    30223025                },
    30233026
    30243027                editImageContent: function() {
     
    50945097                        'click .close':                   'removeFromLibrary',
    50955098                        'click .check':                   'checkClickHandler',
    50965099                        'click a':                        'preventDefault',
     5100                        'keydown .close':                 'removeFromLibrary',
    50975101                        'keydown':                        'toggleSelectionHandler'
    50985102                },
    50995103
     
    55355539                 * @param {Object} event
    55365540                 */
    55375541                removeFromLibrary: function( event ) {
     5542                        // Catch enter and space events
     5543                        if ( 'keydown' === event.type && 13 !== event.keyCode && 32 !== event.keyCode ) {
     5544                                return;
     5545                        }
     5546
    55385547                        // Stop propagation so the model isn't selected.
    55395548                        event.stopPropagation();
    55405549
     
    62256234                        });
    62266235
    62276236                        this.listenTo( this.controller, 'toggle:upload:attachment', _.bind( this.toggleUploader, this ) );
    6228 
     6237                        this.controller.on( 'edit:selection', this.editSelection );
    62296238                        this.createToolbar();
    62306239                        if ( this.options.sidebar ) {
    62316240                                this.createSidebar();
     
    62446253
    62456254                        this.collection.on( 'add remove reset', this.updateContent, this );
    62466255                },
     6256
     6257                editSelection: function( modal ) {
     6258                        modal.$el.find( '.media-button-backToLibrary' ).focus();
     6259                },
     6260
    62476261                /**
    62486262                 * @returns {wp.media.view.AttachmentsBrowser} Returns itself to allow chaining
    62496263                 */