Changeset 26287

Timestamp:

11/20/2013 11:11:01 AM (10 years ago)

Author:

SergeyBiryukov

Message:

Fix JSHint errors in widgets.js.

props jorbin.
fixes #26127.

File:

• trunk/src/wp-admin/js/widgets.js (modified) (13 diffs)

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

@@ -29 +29 @@
 
         $(document.body).bind('click.widgets-toggle', function(e){
-            var target = $(e.target), css = { 'z-index': 100 }, widget, inside, w;
+            var target = $(e.target),
+                css = { 'z-index': 100 },
+                widget, inside, targetWidth, widgetWidth, margin;
 
             if ( target.parents('.widget-top').length && ! target.parents('#available-widgets').length ) {
@@ -69 +71 @@
         sidebars.children('.widget').each(function() {
             wpWidgets.appendTitle(this);
-            if ( $('p.widget-error', this).length )
+            if ( $('p.widget-error', this).length ) {
                 $('a.widget-action', this).click();
+            }
         });
 
@@ -85 +88 @@
             },
             stop: function() {
-                if ( rem )
+                if ( rem ) {
                     $(rem).hide();
+                }
 
                 rem = '';
@@ -103 +107 @@
             },
             stop: function(e,ui) {
-                if ( ui.item.hasClass('ui-draggable') && ui.item.data('draggable') )
+                if ( ui.item.hasClass('ui-draggable') && ui.item.data('draggable') ) {
                     ui.item.draggable('destroy');
+                }
 
                 if ( ui.item.hasClass('deleting') ) {
@@ -140 +145 @@
                 var sender = $(ui.sender);
 
-                if ( !$(this).is(':visible') || this.id.indexOf('orphaned_widgets') > -1 )
+                if ( ! $(this).is(':visible') || this.id.indexOf('orphaned_widgets') > -1 ) {
                     sender.sortable('cancel');
+                }
 
                 if ( sender.attr('id').indexOf('orphaned_widgets') > -1 && !sender.children('.widget').length ) {
@@ -162 +168 @@
                 $('div.widget-placeholder').hide();
 
-                if ( ui.draggable.hasClass('ui-sortable-helper') )
+                if ( ui.draggable.hasClass('ui-sortable-helper') ) {
                     $('#removing-widget').show().children('span')
                     .html( ui.draggable.find('div.widget-title').children('h4').html() );
+                }
             },
             out: function(e,ui) {
@@ -207 +214 @@
                     selectSidebar.find('.widgets-chooser-selected').removeClass( 'widgets-chooser-selected' );
                     $(this).addClass( 'widgets-chooser-selected' );
-                } )
+                } );
             }
         });
@@ -237 +244 @@
 
     saveOrder : function(sb) {
-        if ( sb )
+        if ( sb ) {
             $('#' + sb).closest('div.widgets-holder-wrap').find('.spinner').css('display', 'inline-block');
+        }
 
         var a = {
@@ -247 +255 @@
 
         $('div.widgets-sortables').each( function() {
-            if ( $(this).sortable )
+            if ( $(this).sortable ) {
                 a['sidebars[' + $(this).attr('id') + ']'] = $(this).sortable('toArray').join(',');
+            }
         });
 
@@ -267 +276 @@
         };
 
-        if ( del )
+        if ( del ) {
             a.delete_widget = 1;
+        }
 
         data += '&' + $.param(a);
@@ -276 +286 @@
 
             if ( del ) {
-                if ( !$('input.widget_number', widget).val() ) {
+                if ( ! $('input.widget_number', widget).val() ) {
                     id = $('input.widget-id', widget).val();
                     $('#available-widgets').find('input.widget-id').each(function(){
-                        if ( $(this).val() === id )
+                        if ( $(this).val() === id ) {
                             $(this).closest('div.widget').show();
+                        }
                     });
                 }
@@ -300 +311 @@
                 }
             }
-            if ( order )
+            if ( order ) {
                 wpWidgets.saveOrder();
+            }
         });
     },
@@ -308 +320 @@
         var title = $('input[id*="-title"]', widget).val() || '';
 
-        if ( title )
+        if ( title ) {
             title = ': ' + title.replace(/<[^<>]+>/g, '').replace(/</g, '&lt;').replace(/>/g, '&gt;');
+        }
 
         $(widget).children('.widget-top').children('.widget-title').children()