-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Convert cloud account config to Lit #10350
Conversation
src/components/ha-alert.ts
Outdated
@@ -41,8 +43,12 @@ class HaAlert extends LitElement { | |||
|
|||
@property({ type: Boolean }) public dismissable = false; | |||
|
|||
@property({ type: Boolean }) public ripple = false; |
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 do we need this one? Can't we imply this if it's dismissable or has an action-text ?
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.
Don't forget to also add it to the gallery if we're sticking with a new attr.
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.
You had a ripple effect on the notification in the cloud login, just copied that logic. It has nothing to do with dismissable, it will just show a ripple effect for a second when shown 🤷♂️
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'm totally fine with removing that ripple effect btw :-)
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.
Ah this is a ripple on open. Yeah let's remove it because we have a background color to draw attention.
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.
🎉
58bea33
to
ddd89fa
Compare
Proposed change
Type of change
Example configuration
Additional information
Checklist
If user exposed functionality or configuration variables are added/changed: