Skip to content

Commit 5f893aa

Browse files
authored
Merge pull request #538 from kielni/configure-close-icon
Close icon can now be set as an option.
2 parents 98f68ac + c23a66a commit 5f893aa

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

src/jquery.multiselect.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@
3636
noneSelectedText: 'Select options',
3737
selectedText: '# selected',
3838
selectedList: 0,
39+
closeIcon: 'ui-icon-circle-close',
3940
show: null,
4041
hide: null,
4142
autoOpen: false,
@@ -88,7 +89,7 @@
8889
return '';
8990
}
9091
})
91-
.append('<li class="ui-multiselect-close"><a href="#" class="ui-multiselect-close"><span class="ui-icon ui-icon-circle-close"></span></a></li>')
92+
.append('<li class="ui-multiselect-close"><a href="#" class="ui-multiselect-close"><span class="ui-icon '+o.closeIcon+'"></span></a></li>')
9293
.appendTo(header),
9394

9495
checkboxContainer = (this.checkboxContainer = $('<ul />'))

tests/unit/options.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -340,5 +340,11 @@
340340

341341
el.multiselect("destroy");
342342
});
343-
343+
test("closeIcon", function(){
344+
expect(1);
345+
var icon = "ui-icon-search";
346+
el = $("select").multiselect({ autoOpen:true, closeIcon:icon });
347+
equals(menu().find(".ui-multiselect-close").find("."+icon).length, 1);
348+
el.multiselect("destroy");
349+
});
344350
})(jQuery);

0 commit comments

Comments
 (0)