Changeset 26067

Timestamp:

11/09/2013 09:28:27 PM (11 years ago)

Author:

nacin

Message:

Avoid JSHint errors in color-picker.js. One var, single quotes, add missing semicolon, declare wpColorPickerL10n as global.

props jorbin.
fixes #25877.

File:

• trunk/src/wp-admin/js/color-picker.js (modified) (7 diffs)

trunk/src/wp-admin/js/color-picker.js

@@ +1 @@
+/* global wpColorPickerL10n:true */
 ( function( $, undef ){
 
@@ -5 +6 @@
         _after = '<div class="wp-picker-holder" />',
         _wrap = '<div class="wp-picker-container" />',
-        _button = '<input type="button" class="button button-small hidden" />';
-
+        _button = '<input type="button" class="button button-small hidden" />',
     // jQuery UI Widget constructor
-    var ColorPicker = {
+        ColorPicker = {
         options: {
             defaultColor: false,
@@ -20 +20 @@
             if ( ! $.support.iris )
                 return;
-            var self = this;
-            var el = self.element;
+            var self = this,
+                el = self.element;
 
             $.extend( self.options, el.data() );
@@ -30 +30 @@
             el.addClass( 'wp-color-picker' ).hide().wrap( _wrap );
             self.wrap = el.parent();
-            self.toggler = $( _before ).insertBefore( el ).css( { backgroundColor: self.initialValue } ).attr( "title", wpColorPickerL10n.pick ).attr( "data-current", wpColorPickerL10n.current );
+            self.toggler = $( _before ).insertBefore( el ).css( { backgroundColor: self.initialValue } ).attr( 'title', wpColorPickerL10n.pick ).attr( 'data-current', wpColorPickerL10n.current );
             self.pickerContainer = $( _after ).insertAfter( el );
             self.button = $( _button );
@@ -70 +70 @@
                 // close picker when you click outside it
                 if ( self.toggler.hasClass( 'wp-picker-open' ) )
-                    $( "body" ).on( 'click', { wrap: self.wrap, toggler: self.toggler }, self._bodyListener );
+                    $( 'body' ).on( 'click', { wrap: self.wrap, toggler: self.toggler }, self._bodyListener );
                 else
-                    $( "body" ).off( 'click', self._bodyListener );
+                    $( 'body' ).off( 'click', self._bodyListener );
             });
 
@@ -115 +115 @@
         color: function( newColor ) {
             if ( newColor === undef )
-                return this.element.iris( "option", "color" );
+                return this.element.iris( 'option', 'color' );
 
-            this.element.iris( "option", "color", newColor );
+            this.element.iris( 'option', 'color', newColor );
         },
         //$("#input").wpColorPicker('defaultColor') returns the current default color
@@ -127 +127 @@
             this.options.defaultColor = newDefaultColor;
         }
-    }
+    };
 
     $.widget( 'wp.wpColorPicker', ColorPicker );