WordPress.org

Make WordPress Core

Changeset 37755


Ignore:
Timestamp:
06/21/2016 01:34:19 PM (4 years ago)
Author:
joemcgill
Message:

Media: Restore keyboard navigation of the media grid.

This changes the binding of event listeners in the Attachments Browser
to use on instead of listenTo for the attachment:keydown:arrow and
attachment:details:shift-tab events. The existing listeners broke
when we upgraded Backbone in [36546].

Props adamsilverstein.
Fixes #36900.

Location:
trunk/src/wp-includes/js
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/js/media-views.js

    r37678 r37755  
    40864086
    40874087        // Add keydown listener to the instance of the Attachments view
    4088         this.attachments.listenTo( this.controller, 'attachment:keydown:arrow',     this.attachments.arrowEvent );
    4089         this.attachments.listenTo( this.controller, 'attachment:details:shift-tab', this.attachments.restoreFocus );
     4088        this.controller.on( 'attachment:keydown:arrow',     _.bind( this.attachments.arrowEvent, this.attachments ) );
     4089        this.controller.on( 'attachment:details:shift-tab', _.bind( this.attachments.restoreFocus, this.attachments ) );
    40904090
    40914091        this.views.add( this.attachments );
  • trunk/src/wp-includes/js/media/views/attachments/browser.js

    r37610 r37755  
    355355
    356356        // Add keydown listener to the instance of the Attachments view
    357         this.attachments.listenTo( this.controller, 'attachment:keydown:arrow',     this.attachments.arrowEvent );
    358         this.attachments.listenTo( this.controller, 'attachment:details:shift-tab', this.attachments.restoreFocus );
     357        this.controller.on( 'attachment:keydown:arrow',     _.bind( this.attachments.arrowEvent, this.attachments ) );
     358        this.controller.on( 'attachment:details:shift-tab', _.bind( this.attachments.restoreFocus, this.attachments ) );
    359359
    360360        this.views.add( this.attachments );
Note: See TracChangeset for help on using the changeset viewer.