Trigger the onChange event only when the value has changed. Fixes #118

Author: lcdsantos

public/index.html

@@ -627,6 +627,9 @@ <h4><a href="./demo.html">view more advanced demos and customization →</a></h4
       });
 
       $('select').selectric();
+      $('select').on('change', function() {
+        console.log('Change: ', this, $(this).val());
+      });
     });
 
     (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){

public/jquery.selectric.js

@@ -859,19 +859,19 @@
         _this.state.currValue = _this.state.selectedIdx;
 
         _this.setLabel();
-      }
 
-      if ( _this.state.currValue !== _this.state.selectedIdx && !_this.state.multiple ) {
+        _this.utils.triggerCallback('Change', _this);
+      } else if ( _this.state.currValue !== _this.state.selectedIdx ) {
         // Apply changed value to original select
         _this.$element
           .prop('selectedIndex', _this.state.currValue = _this.state.selectedIdx)
           .data('value', _this.lookupItems[_this.state.selectedIdx].text);
 
         // Change label text
         _this.setLabel();
-      }
 
-      _this.utils.triggerCallback('Change', _this);
+        _this.utils.triggerCallback('Change', _this);
+      }
     },
 
     /**

public/jquery.selectric.min.js

@@ -842,19 +842,19 @@
         _this.state.currValue = _this.state.selectedIdx;
 
         _this.setLabel();
-      }
 
-      if ( _this.state.currValue !== _this.state.selectedIdx && !_this.state.multiple ) {
+        _this.utils.triggerCallback('Change', _this);
+      } else if ( _this.state.currValue !== _this.state.selectedIdx ) {
         // Apply changed value to original select
         _this.$element
           .prop('selectedIndex', _this.state.currValue = _this.state.selectedIdx)
           .data('value', _this.lookupItems[_this.state.selectedIdx].text);
 
         // Change label text
         _this.setLabel();
-      }
 
-      _this.utils.triggerCallback('Change', _this);
+        _this.utils.triggerCallback('Change', _this);
+      }
     },
 
     /**

src/jquery.selectric.js

@@ -19,6 +19,14 @@ describe('events', function() {
     expect(changedFn).toHaveBeenCalled();
   });
 
+  it('should not trigger change event', function() {
+    var notChangedFn = jasmine.createSpy('notChangedFn');
+    select.on('change', notChangedFn);
+    $('.selectric').click();
+    $('.selectric-items').find('li.selected').click();
+    expect(notChangedFn).not.toHaveBeenCalled();
+  });
+
   it('should trigger events', function() {
     var events = jasmine.createSpyObj('events', [
       'beforeInit',