deac/chosen
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Remove condition made unnecessary by poorly written CoffeeScript. The text of an option was never matching this function, so why leave it there?

Browse source
This commit is contained in:
Patrick Filler 2011-11-04 15:48:50 -04:00
parent 948fbcca11
commit f0a349d782
4 changed files with 5 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
chosen/chosen.jquery.js
View file

@ -724,9 +724,7 @@
} else { } else {
text = option.html; text = option.html;
} }
if (result.html !== text) {
result.html(text); result.html(text);
}
this.result_activate(result); this.result_activate(result);
if (option.group_array_index != null) { if (option.group_array_index != null) {
$("#" + this.results_data[option.group_array_index].dom_id).css('display', 'auto'); $("#" + this.results_data[option.group_array_index].dom_id).css('display', 'auto');

2
chosen/chosen.jquery.min.js vendored
View file

File diff suppressed because one or more lines are too long

2
chosen/chosen.proto.min.js vendored
View file

File diff suppressed because one or more lines are too long

3
coffee/chosen.jquery.coffee
View file

@ -415,8 +415,7 @@ class Chosen extends AbstractChosen
else else
text = option.html text = option.html
result.html text if result.html != text result.html(text)
this.result_activate result this.result_activate result
$("#" + @results_data[option.group_array_index].dom_id).css('display', 'auto') if option.group_array_index? $("#" + @results_data[option.group_array_index].dom_id).css('display', 'auto') if option.group_array_index?