-
Notifications
You must be signed in to change notification settings - Fork 4
Updated radio and checkboxes to only apply label styles if a label is present #42
Conversation
Codecov Report
@@ Coverage Diff @@
## master #42 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 13 13
Lines 536 536
Branches 88 88
=====================================
Hits 536 536 Continue to review full report at Codecov.
|
role="radiogroup" | ||
> | ||
<li> | ||
<div id="screenshot-icon-radio" class="sky-switch-icon-group"> |
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.
attributes on new lines
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.
Done
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 thing, otherwise great.
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.
LGTM.
We introduced a bug with this pull request, for context: blackbaud/skyux-theme#62