Changeset 32462

Timestamp:

05/08/2015 08:09:58 PM (11 years ago)

Author:

wonderboymusic

Message:

When creating an EmbedLink view, don't add a spinner - it already exists in Embed.
Use the value of wp.media.controller.Embed.sensitivity as the sensitivity value for the debounced method in EmbedLink to ensure that the spinner toggles don't stutter or overlap.

Fixes #32036.

Location:

trunk/src/wp-includes/js

Files:

• media-views.js (modified) (3 diffs)
• media/controllers/embed.js (modified) (1 diff)
• media/views/embed/link.js (modified) (2 diffs)

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

@@ -504 +504 @@
 
         // The amount of time used when debouncing the scan.
-        sensitivity: 200,
+        sensitivity: 400,
 
         initialize: function(options) {
@@ -4534 +4534 @@
 
         initialize: function() {
-                this.spinner = $('<span class="spinner" />');
-                this.$el.append( this.spinner[0] );
                 this.listenTo( this.model, 'change:url', this.updateoEmbed );
         },
@@ -4552 +4550 @@
 
                 this.fetch();
-        }, 600 ),
+        }, wp.media.controller.Embed.sensitivity ),
 
         fetch: function() {

trunk/src/wp-includes/js/media/controllers/embed.js

@@ -40 +40 @@
 
         // The amount of time used when debouncing the scan.
-        sensitivity: 200,
+        sensitivity: 400,
 
         initialize: function(options) {

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

@@ -18 +18 @@
 
         initialize: function() {
-                this.spinner = $('<span class="spinner" />');
-                this.$el.append( this.spinner[0] );
                 this.listenTo( this.model, 'change:url', this.updateoEmbed );
         },
@@ -36 +34 @@
 
                 this.fetch();
-        }, 600 ),
+        }, wp.media.controller.Embed.sensitivity ),
 
         fetch: function() {