added placeholder attribute support for compact multiple selects

w3co · May 27, 2015 · 34f785c · 34f785c
1 parent 39a39be
commit 34f785c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/index.html b/index.html
@@ -215,8 +215,7 @@ <h2>&lt;select&gt;</h2>
 					<option>Option 22</option>
 					<option>Option 23</option>
 				</select>
-				<select multiple data-jcf='{"wrapNative": false, "wrapNativeOnMobile": false, "useCustomScroll": false, "multipleCompactStyle": true}'>
-					<option selected value="v1">Compact multiple select</option>
+				<select multiple placeholder="Compact multiple select" data-jcf='{"wrapNative": false, "wrapNativeOnMobile": false, "useCustomScroll": false, "multipleCompactStyle": true}'>
 					<option value="v2">Option 1</option>
 					<option value="v3">Option 2</option>
 					<option value="v4">Option 3</option>

diff --git a/js/jcf.select.js b/js/jcf.select.js
@@ -388,14 +388,18 @@
 				selectedOption = this.realElement.prop('options')[selectedIndex],
 				selectedOptionImage = selectedOption ? selectedOption.getAttribute('data-image') : null,
 				selectedOptionText = '',
-				selectedOptionClasses;
+				selectedOptionClasses,
+				self = this;
 
 			if (this.realElement.prop('multiple')) {
 				$.each(this.realElement.prop('options'), function(index, option) {
 					if (option.selected) {
 						selectedOptionText += (selectedOptionText ? ', ' : '') + option.innerHTML;
 					}
 				});
+				if (!selectedOptionText) {
+					selectedOptionText = self.realElement.attr('placeholder') || '';
+				}
 				this.selectText.removeAttr('class').html(selectedOptionText);
 			} else if (!selectedOption) {
 				if (this.selectImage) {