Fix #13: click current option item should not fire onchange event.

erik168 · Jul 20, 2012 · 9d92174 · 9d92174
1 parent 566b70e
commit 9d92174
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/esui/Select.js b/src/esui/Select.js
@@ -488,15 +488,15 @@ esui.Select.prototype = {
      * @param {HTMLElement} item 选项
      */
     _itemClickHandler: function ( item ) {
-        var index = item.getAttribute( 'index' );
+        var index = parseInt( item.getAttribute( 'index' ), 10 );
         var disabled = item.getAttribute( 'dis' );
 
         if ( disabled == 1 ) {
             return;
         }
 
         this.hideLayer();
-        this.setSelectedIndex( parseInt( index, 10 ), true );
+        this.setSelectedIndex( index, index != this.selectedIndex );
     },
 
     /**