-
-
Notifications
You must be signed in to change notification settings - Fork 307
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
feat(chips): add alarm control panel chip #131
Conversation
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.
Very cool ! 👍
Thanks for the translations and the improvements !
Also remove old log and enable color even for not active alarm panel (ie disarmed).
For the translation, if the key is not present in the file, it will fallback to english so it's not a problem 🙂 |
@@ -92,7 +92,7 @@ alarm_control_panel: | |||
- alarm_control_panel.alarm_panel_1 | |||
- alarm_control_panel.alarm_panel_2 | |||
disarm: | |||
service: alarm_control_panel.alarm_arm_home | |||
service: alarm_control_panel.alarm_disarm |
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.
👍
@@ -105,7 +105,7 @@ export class ChipsCardEditorChips extends LitElement { | |||
${CHIP_LIST.map( | |||
(chip) => html` | |||
<paper-item data-type="${chip}" } |
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.
I would recommend moving away from Paper Drop down menus. I know this was not changed in this PR but something to think about.
HA frontend is moving out all paper Drop downs and that means the component will no longer be loaded. We have moved to mwc-select
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.
Thanks for pointing that, I'll create a tech issue.
Add alarm control panel that displays icon and color like the card.
Also added chip name translation, if PR is validated, we have to update translations in #130
fix #105