-
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
Changed button to text #4067
Changed button to text #4067
Conversation
src/resources/styles.ts
Outdated
.paper-dialog-buttons .warning.left-align { | ||
margin-right: auto; | ||
} |
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.
Move this to hui-edit-views
static get styles()
, there is a .delete
class there that was used for the button, should clean that up and replace with the new styles.
src/translations/en.json
Outdated
@@ -502,6 +502,7 @@ | |||
"common": { | |||
"loading": "Loading", | |||
"cancel": "Cancel", | |||
"delete_view": "Delete View", |
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.
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.
This should not be in common
but in the section of lovelace and the views, there actually already is a key for delete
there.
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.
@bramkragten I didn't realize those existed in en.json.
@bramkragten I have updated the PR with the styles being moved, and changing what the translation values are. |
Updated the trashcan icon to delete text.
Addresses feature #4027