Browse Source

Fixes #2806 issue with trim in dropdown

pull/2755/merge
Jack Lukic 9 years ago
parent
commit
b615696209
2 changed files with 4 additions and 3 deletions
  1. 1
      RELEASE-NOTES.md
  2. 6
      src/definitions/modules/dropdown.js

1
RELEASE-NOTES.md

@ -27,6 +27,7 @@
- **Transition** - Transition callbacks now all have the correct `this` set. #2758
- **Form / Input** - Fixes `::placeholder` text color for `ui error input`, modifies form error placeholder color to distinguish from form value error color #2786
- **Form** - Date input and other special input in chrome now are the same height as normal input (adds custom vendor shadow dom styling) #2704
- **Dropdown** - Remove use of `trim` which causes issues IE 11 and below #2806
**Additional Bugs**
- **Build Tools** - Fixes issue on `win` platform where packaged theme would not correctly update when using watch due to regExp not matching windows path separators.

6
src/definitions/modules/dropdown.js

@ -1509,8 +1509,8 @@ $.fn.dropdown = function(parameters) {
return ($choice.data(metadata.text) !== undefined)
? $choice.data(metadata.text)
: (preserveHTML)
? $choice.html().trim()
: $choice.text().trim()
? $.trim($choice.html())
: $.trim($choice.text())
;
}
},
@ -1522,7 +1522,7 @@ $.fn.dropdown = function(parameters) {
return ($choice.data(metadata.value) !== undefined)
? $choice.data(metadata.value)
: (typeof choiceText === 'string')
? choiceText.toLowerCase().trim()
? $.trim(choiceText.toLowerCase())
: choiceText
;
},

Loading…
Cancel
Save