Ticket #30251: 30251.diff

src/wp-admin/js/customize-controls.js

@@ -131 +131 @@
          * @augments wp.customize.Class
          */
         Container = api.Class.extend({
-                defaultActiveArguments: { duration: 'fast' },
-                defaultExpandedArguments: { duration: 'fast' },
+                defaultActiveArguments: { duration: 'fast', completeCallback: $.noop },
+                defaultExpandedArguments: { duration: 'fast', completeCallback: $.noop },
 
                 initialize: function ( id, options ) {
                         var container = this;
@@ -217 +217 @@
                  */
                 onChangeActive: function ( active, args ) {
                         var duration = ( 'resolved' === api.previewer.deferred.active.state() ? args.duration : 0 );
-                        if ( active ) {
+                        if ( ! $.contains( document, this.container ) ) {
+                                // jQuery.fn.slideUp is not hiding an element if it is not in the DOM
+                                this.container.toggle( active );
+                                args.completeCallback();
+                        } else if ( active ) {
                                 this.container.stop( true, true ).slideDown( duration, args.completeCallback );
                         } else {
                                 this.container.stop( true, true ).slideUp( duration, args.completeCallback );
@@ -640 +644 @@
          * @augments wp.customize.Class
          */
         api.Control = api.Class.extend({
-                defaultActiveArguments: { duration: 'fast' },
+                defaultActiveArguments: { duration: 'fast', completeCallback: $.noop },
 
                 initialize: function( id, options ) {
                         var control = this,
@@ -781 +785 @@
                  * @param {Object} args  merged on top of this.defaultActiveArguments
                  */
                 onChangeActive: function ( active, args ) {
-                        if ( active ) {
+                        if ( ! $.contains( document, this.container ) ) {
+                                // jQuery.fn.slideUp is not hiding an element if it is not in the DOM
+                                this.container.toggle( active );
+                                args.completeCallback();
+                        } else if ( active ) {
                                 this.container.slideDown( args.duration, args.completeCallback );
                         } else {
                                 this.container.slideUp( args.duration, args.completeCallback );

src/wp-admin/js/customize-widgets.js

@@ -405 +405 @@
          */
         api.Widgets.WidgetControl = api.Control.extend({
                 defaultExpandedArguments: {
-                        duration: 'fast'
+                        duration: 'fast',
+                        completeCallback: $.noop
                 },
 
                 initialize: function ( id, options ) {