New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace checkboxes in list items with check-list-item
#11610
Conversation
src/components/ha-check-list-item.ts
Outdated
import { CheckListItem } from "@material/mwc-list/mwc-check-list-item"; | ||
import { customElement } from "lit/decorators"; | ||
|
||
const styles = [ |
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.
Why define above and not use static styles = […]
?
src/components/ha-radio.ts
Outdated
@@ -1,11 +1,19 @@ | |||
import { Radio } from "@material/mwc-radio"; |
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.
Should we go all fancy and import just the base and the styles? like this
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.
👍🏼 yes
Co-authored-by: Paulus Schoutsen <balloob@gmail.com>
Breaking change
Proposed change
Type of change
Example configuration
Additional information
Checklist
If user exposed functionality or configuration variables are added/changed: