-
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
L10N config-entries #1718
L10N config-entries #1718
Conversation
src/translations/en.json
Outdated
"manuf": "by {manufacturer}", | ||
"hub": "Connected via", | ||
"firmware": "Firmware: {version}", | ||
"device_unavailable": "(device unavailable)", |
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.
Let's move the parentheses out of the translation.
@@ -716,8 +716,8 @@ | |||
"manuf": "by {manufacturer}", | |||
"hub": "Connected via", | |||
"firmware": "Firmware: {version}", | |||
"device_unavailable": "(device unavailable)", | |||
"entity_unavailable": "(entity unavailable)" | |||
"device_unavailable": "device unavailable", |
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.
Now you will need to add the parentheses back to the layout
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.
ok to merge when parentheses have been added back to the layout.
@@ -19,7 +19,7 @@ function computeEntityName(hass, entity) { | |||
/* | |||
* @appliesMixin EventsMixin | |||
*/ | |||
class HaDeviceCard extends EventsMixin(PolymerElement) { | |||
class HaCeEntitiesCard extends EventsMixin(PolymerElement) { |
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.
We forgot to apply the localize mix in here.
Fix here: #1757
Now i need to consider what I want to do for 0.80 😢
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.
(verified that this was the only one in this PR)
Partial resolve #1593