Merge pull request #657 from ehynds/Issue#656

Fixes #656, Fixes #590

Author: mlh758

src/jquery.multiselect.js

@@ -43,7 +43,8 @@
       multiple: true,
       position: {},
       appendTo: "body",
-      menuWidth:null
+      menuWidth:null,
+      selectedListSeparator: ', '
     },
 
     _create: function() {
@@ -242,7 +243,7 @@
         if($.isFunction(o.selectedText)) {
           value = o.selectedText.call(this, numChecked, $inputs.length, $checked.get());
         } else if(/\d/.test(o.selectedList) && o.selectedList > 0 && numChecked <= o.selectedList) {
-          value = $checked.map(function() { return $(this).next().text(); }).get().join(', ');
+          value = $checked.map(function() { return $(this).next().text(); }).get().join(o.selectedListSeparator);
         } else {
           value = o.selectedText.replace('#', numChecked).replace('#', $inputs.length);
         }
@@ -754,6 +755,11 @@
           break;
         case 'position':
           this.position();
+          break;
+        case 'selectedListSeparator':
+          this.options[key] = value;
+          this.button[0].defaultValue = this.update();
+          break;
       }
 
       $.Widget.prototype._setOption.apply(this, arguments);

tests/unit/options.js

@@ -347,4 +347,19 @@
 		equals(menu().find(".ui-multiselect-close").find("."+icon).length, 1);
 		el.multiselect("destroy");
 	});
+    test("selectedListSeparator", function(){
+        expect(3);
+        el = $("select").multiselect({ selectedListSeparator: "<br/>", selectedList: 15 });
+        el.multiselect("checkAll");
+        var text = $(button()).text();
+        var matched = [];
+        equals(text.indexOf(","), -1, "There are no commas in the button text");
+        matched = text.match(/\<br\/\>/g);
+        equals(matched.length, 8, "The 9 selected values are joined by <br> tags");
+        el.multiselect("option", "selectedListSeparator", ", ");
+        text = $(button()).text();
+        matched = text.match(/\,/g)
+        equals(matched.length, 8, "The 9 selected values are joined by commas again after calling the option method");
+        el.multiselect("destroy");
+    });
 })(jQuery);