Changeset 29558

Timestamp:

08/20/2014 07:59:59 PM (5 years ago)

Author:

wonderboymusic

Message:

Cleanup after [29220]:

media.controller.Library is State/Backbone Model and should not touch views.

See #23560.

File:

• trunk/src/wp-includes/js/media-views.js (modified) (3 diffs)

trunk/src/wp-includes/js/media-views.js

@@ -754 +754 @@
             this.get('selection').add( attachment );
 
-            // Set focus back to where it goes when an attachment is selected.
-            $( '.attachments-browser .attachments .attachment' ).first().focus();
+            this.frame.trigger( 'library:selection:add' );
         },
 
@@ -5257 +5256 @@
             this.collection.on( 'reset', this.render, this );
 
+            this.listenTo( this.controller, 'library:selection:add',    this.attachmentFocus );
             this.listenTo( this.controller, 'attachment:keydown:arrow', this.arrowEvent );
 
@@ -5277 +5277 @@
             // attachments get proper width applied.
             _.defer( this.setColumns, this );
+        },
+
+        attachmentFocus: function() {
+            this.$( 'li:first' ).focus();
         },