Merge pull request #4 from kumardeo/fix-onchange

Fix `onChange` firing before changing the color

Author: yairEO

README.md

@@ -101,9 +101,9 @@ const cPicker = new ColorPicker({
 
   // buttons text & icon
   buttons: {
-    undo   : { icon: '↶', title: 'Undo' },
-    add    : { icon: '+', title: 'Add to Swatches' },
-    format : { icon: '⇆', title: 'Switch Color Format' }
+    undo   : { icon: '↶', title: 'Undo' },
+    format : { icon: '⇆', title: 'Switch Color Format' },
+    add    : { icon: '+', title: 'Add to Swatches' }
   },
 
   // fired rapidly, on every range slider movement or any color change

dist/color-picker.es.js

@@ -68,7 +68,7 @@ ColorPicker.prototype = {
     return result
   },
 
-  swithFormat(){
+  switchFormat(){
     const _cf = this.colorFormat
 
     switch( _cf ){

dist/color-picker.es.js.map

@@ -4,11 +4,11 @@ export default {
   onChange: _=>_,
   buttons: {
     undo: {
-      icon: '↶',
+      icon: '↶',
       title: 'Undo'
     },
     format: {
-      icon: '⇆',
+      icon: '⇆',
       title: 'Switch Color Format'
     },
     add: {

dist/color-picker.iife.js

@@ -77,7 +77,7 @@ function onButtonClick(e){
   //   case
   // }
   if( name == 'format' )
-    this.swithFormat()
+    this.switchFormat()
 
   else if( name == 'undo' )
     this.history.undo()
@@ -89,7 +89,7 @@ function onButtonClick(e){
     this.removeSwatch(pNode, pNode.title)
 
   else if( classList.contains('color-picker__swatch') && title ){
-    this.history.last = this.color
     this.setColor( this.getHSLA(title) )
+    this.history.last = this.color
   }
 }