-
Notifications
You must be signed in to change notification settings - Fork 2.1k
fix(select): Fix floating label for pre-selected option #2306
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2306 +/- ##
==========================================
+ Coverage 99.07% 99.07% +<.01%
==========================================
Files 99 99
Lines 4004 4007 +3
Branches 515 516 +1
==========================================
+ Hits 3967 3970 +3
Misses 37 37
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One super tiny nit - otherwise LGTM!
demos/select.html
Outdated
setSelectedCb.addEventListener('change', function() { | ||
select.selectedIndex = this.checked ? 0 : -1; | ||
}); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Remove blank line
demos/select.html
Outdated
@@ -196,6 +204,8 @@ <h2 class="mdc-typography--title">Select box</h2> | |||
var rtlCb = document.getElementById('rtl'); | |||
var alternateColorsCb = document.getElementById('alternate-colors'); | |||
var disabledCb = document.getElementById('disabled'); | |||
var setSelectedCb = document.getElementById('set-selected-checkbox'); | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Remove blank line
0984f22
to
a8206cb
Compare
Fixes: #2282
Added tests and demo to check for regression.