Merge pull request #2225 from davidfstr/fix_empty_placeholder
Allow an empty string to be used as the placeholder title.
This commit is contained in:
commit
60d74da872
@ -2241,7 +2241,7 @@ the specific language governing permissions and limitations under the Apache Lic
|
|||||||
|
|
||||||
isPlaceholderOptionSelected: function() {
|
isPlaceholderOptionSelected: function() {
|
||||||
var placeholderOption;
|
var placeholderOption;
|
||||||
if (!this.getPlaceholder()) return false; // no placeholder specified so no option should be considered
|
if (this.getPlaceholder() === undefined) return false; // no placeholder specified so no option should be considered
|
||||||
return ((placeholderOption = this.getPlaceholderOption()) !== undefined && placeholderOption.prop("selected"))
|
return ((placeholderOption = this.getPlaceholderOption()) !== undefined && placeholderOption.prop("selected"))
|
||||||
|| (this.opts.element.val() === "")
|
|| (this.opts.element.val() === "")
|
||||||
|| (this.opts.element.val() === undefined)
|
|| (this.opts.element.val() === undefined)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user