Changeset 22660

Timestamp:

11/19/2012 04:41:57 AM (14 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;
                 }