Skip to content

Commit

Permalink
No need to refresh the select for IE >= 8
Browse files Browse the repository at this point in the history
  • Loading branch information
vicb committed Nov 26, 2010
1 parent 69ff6a5 commit 3829792
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
4 changes: 4 additions & 0 deletions history.md
@@ -1,5 +1,9 @@
# bsmSelect #

v1.4.1 - 2010-11-26

* Do not force refresh the select for IE > 7 to remove flickering (this bug seems to be gone from IE8)

v1.4.0 - 2010-09-05

* API break: animate & highlight options (the compatibility plugin might be used for backward compatibility),
Expand Down
6 changes: 3 additions & 3 deletions js/jquery.bsmselect.js
Expand Up @@ -8,7 +8,7 @@
*
* Dual licensed under the MIT (MIT-LICENSE.txt) and GPL (GPL-LICENSE.txt) licenses.
*
* bsmSelect version: v1.4.0 - 2010-09-05
* bsmSelect version: v1.4.1 - 2010-11-26
*/

(function($) {
Expand Down Expand Up @@ -192,7 +192,7 @@
.removeAttr('selected')
.attr('disabled', 'disabled')
.toggle(!this.options.hideWhenAdded);
if ($.browser.msie) { this.$select.hide().show(); } // this forces IE to update display
if ($.browser.msie && $.browser.version < 8) { this.$select.hide().show(); } // this forces IE to update display
},

/**
Expand All @@ -204,7 +204,7 @@
$bsmOpt.removeClass(this.options.optionDisabledClass)
.removeAttr('disabled')
.toggle(!this.options.hideWhenAdded);
if ($.browser.msie) { this.$select.hide().show(); } // this forces IE to update display
if ($.browser.msie && $.browser.version < 8) { this.$select.hide().show(); } // this forces IE to update display
},

/**
Expand Down

0 comments on commit 3829792

Please sign in to comment.