Changeset 22660

Timestamp:

11/19/2012 04:41:57 AM (12 years ago)

Author:

koopersmith

Message:

Media: Allow subviews to be inserted at a specific index. see #21390.

File:

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

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

@@ -592 +592 @@
             add      = options && options.add;
             existing = this.get( selector );
+            next     = views;
             method   = add ? 'attach' : 'replace';
 
-            if ( ! add && existing ) {
-                this.unset( selector );
-                _.invoke( existing, 'dispose' );
+            if ( existing ) {
+                if ( add ) {
+                    if ( _.isUndefined( options.at ) )
+                        next = existing.concat( views );
+                    else
+                        next = existing.splice.apply( existing, [ options.at, 0 ].concat( views ) );
+                } else {
+                    this.unset( selector );
+                    _.invoke( existing, 'dispose' );
+                }
             }
 
-            this._views[ selector ] = add && existing ? existing.concat( views ) : views;
+            this._views[ selector ] = next;
 
             $selector = selector ? this.view.$( selector ) : this.view.$el;
@@ -611 +619 @@
             }, this );
 
-            this[ method ]( $selector, els );
-            return this;
-        },
-
-        add: function( selector, views ) {
-            return this.set( selector, views, { add: true });
+            this[ method ]( $selector, els, options );
+            return this;
+        },
+
+        add: function( selector, views, options ) {
+            return this.set( selector, views, _.extend({ add: true }, options ) );
         },
 
@@ -662 +670 @@
         },
 
-        attach: function( $target, els ) {
-            $target.append( els );
+        attach: function( $target, els, options ) {
+            var at = options && options.at,
+                $children;
+
+            if ( _.isNumber( at ) && ($children = $target.children()).length > at )
+                $children.eq( at ).before( els );
+            else
+                $target.append( els );
+
             return this;
         }