Changeset 37804

Timestamp:

06/21/2016 02:48:13 PM (7 years ago)

Author:

swissspidy

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].

Merge of [37755] to the 4.5 branch.

Props adamsilverstein.
Fixes #36900.

Location:

branches/4.5

Files:

• . (modified) (1 prop)
• src/wp-includes/js/media-views.js (modified) (1 diff)
• src/wp-includes/js/media/views/attachments/browser.js (modified) (1 diff)

branches/4.5/src/wp-includes/js/media-views.js

@@ -4086 +4086 @@
 
         // Add keydown listener to the instance of the Attachments view
-        this.attachments.listenTo( this.controller, 'attachment:keydown:arrow',     this.attachments.arrowEvent );
-        this.attachments.listenTo( this.controller, 'attachment:details:shift-tab', this.attachments.restoreFocus );
+        this.controller.on( 'attachment:keydown:arrow',     _.bind( this.attachments.arrowEvent, this.attachments ) );
+        this.controller.on( 'attachment:details:shift-tab', _.bind( this.attachments.restoreFocus, this.attachments ) );
 
         this.views.add( this.attachments );

branches/4.5/src/wp-includes/js/media/views/attachments/browser.js

@@ -355 +355 @@
 
         // Add keydown listener to the instance of the Attachments view
-        this.attachments.listenTo( this.controller, 'attachment:keydown:arrow',     this.attachments.arrowEvent );
-        this.attachments.listenTo( this.controller, 'attachment:details:shift-tab', this.attachments.restoreFocus );
+        this.controller.on( 'attachment:keydown:arrow',     _.bind( this.attachments.arrowEvent, this.attachments ) );
+        this.controller.on( 'attachment:details:shift-tab', _.bind( this.attachments.restoreFocus, this.attachments ) );
 
         this.views.add( this.attachments );