A few "cosmetic bugs" in the user interface
Moderator: leecollings
- PeGe
- Posts: 25
- Joined: Tuesday 31 January 2017 14:21
- Target OS: Raspberry Pi / ODroid
- Domoticz version: V3.6328
- Location: Sollentuna, Sweden
- Contact:
A few "cosmetic bugs" in the user interface
First; I noticed that the display text "Humidity %" always appears near the right hand side of the temperature graphs, even if the affected sensor is of type 'temperature-only'. However, there is apparently some conditional logic that automatically removes the "numeric scale percentage notation" from these graphs. So the same conditional logic would just need to be applied to the "Humidity %" string as well.
Second; It seems like the left hand side display text "degrees Celsius" in the same graphs is not translatable. Even though the "Celsius" part of the string is international, the word "degrees" is not...
Third; The string "Last seen" is currently not part of the translatable display texts, as I found it to be hard coded in the CSS. (Easy to fix locally by a dirty CSS hack, but still...)
Fourth; To my understanding this has already been mentioned elsewhere, but I still want to point it out: The word "Language" on the setup page.
I deem these flaws as merely cosmetic, so the priority can definitely be considered low. But it would be nice to have them sorted at some point in time.
/P-G
Second; It seems like the left hand side display text "degrees Celsius" in the same graphs is not translatable. Even though the "Celsius" part of the string is international, the word "degrees" is not...
Third; The string "Last seen" is currently not part of the translatable display texts, as I found it to be hard coded in the CSS. (Easy to fix locally by a dirty CSS hack, but still...)
Fourth; To my understanding this has already been mentioned elsewhere, but I still want to point it out: The word "Language" on the setup page.
I deem these flaws as merely cosmetic, so the priority can definitely be considered low. But it would be nice to have them sorted at some point in time.
/P-G
- gizmocuz
- Posts: 2350
- Joined: Thursday 11 July 2013 18:59
- Target OS: Raspberry Pi / ODroid
- Domoticz version: beta
- Location: Top of the world
- Contact:
Re: A few "cosmetic bugs" in the user interface
True, could you submit a git pull request for this ?
Specially the '3th' statement, i tried, but failed...
We need a CSS/Javascript hack, as the 'Last seen' is inside the CSS
Specially the '3th' statement, i tried, but failed...
We need a CSS/Javascript hack, as the 'Last seen' is inside the CSS
Quality outlives Quantity!
- gizmocuz
- Posts: 2350
- Joined: Thursday 11 July 2013 18:59
- Target OS: Raspberry Pi / ODroid
- Domoticz version: beta
- Location: Top of the world
- Contact:
Re: A few "cosmetic bugs" in the user interface
#1 will be solved in a next beta, adding 'showEmpty: false' to the yAxis solved the problem
#4, not sure what you mean with this
#4, not sure what you mean with this
Quality outlives Quantity!
- PeGe
- Posts: 25
- Joined: Tuesday 31 January 2017 14:21
- Target OS: Raspberry Pi / ODroid
- Domoticz version: V3.6328
- Location: Sollentuna, Sweden
- Contact:
Re: A few "cosmetic bugs" in the user interface
Hi gizmocuz,
First let me say I admire your commitment towards constantly improving both the core software as well as the user experience. - Impressive!
Now, to clarify point #4 in my original posting:
/P-G
First let me say I admire your commitment towards constantly improving both the core software as well as the user experience. - Impressive!
Now, to clarify point #4 in my original posting:
- In the top left hand portion of the 'Settings' page, under 'System settings' and 'User interface', you find the "Language" selection. No matter what language you currently use, that specific attribute name string, i.e. the word "Language", always stays the same.
/P-G
- gizmocuz
- Posts: 2350
- Joined: Thursday 11 July 2013 18:59
- Target OS: Raspberry Pi / ODroid
- Domoticz version: beta
- Location: Top of the world
- Contact:
Re: A few "cosmetic bugs" in the user interface
That is correct, and also intended.
If someone puts my domoticz in Arabic language, i still want to understand it. This string will not be translated.
Do you have some code for #3 maybe ?
If someone puts my domoticz in Arabic language, i still want to understand it. This string will not be translated.
Do you have some code for #3 maybe ?
Quality outlives Quantity!
- PeGe
- Posts: 25
- Joined: Tuesday 31 January 2017 14:21
- Target OS: Raspberry Pi / ODroid
- Domoticz version: V3.6328
- Location: Sollentuna, Sweden
- Contact:
Re: A few "cosmetic bugs" in the user interface
OK, I see your point! And it makes sense to leave that wording "as-is".
Let me look into what I can do with #3. As for myself, I simply edited the CSS in my own installation, but that raw quick-and-dirty hack is clearly just a local short-term workaround. However, I recall noticing some other CSS based strings that are actually subjected to translation, so the mechanism is obviously in place already. But I didn't dig deeper into the matter at that time.
EDIT: My mistake above! No mechanism for "locale-aware" CSS seems to be in place. I will check for other options...
I will report back if/when I come up with a permanent solution!
/P-G
Let me look into what I can do with #3. As for myself, I simply edited the CSS in my own installation, but that raw quick-and-dirty hack is clearly just a local short-term workaround. However, I recall noticing some other CSS based strings that are actually subjected to translation, so the mechanism is obviously in place already. But I didn't dig deeper into the matter at that time.
EDIT: My mistake above! No mechanism for "locale-aware" CSS seems to be in place. I will check for other options...
I will report back if/when I come up with a permanent solution!
/P-G
- PeGe
- Posts: 25
- Joined: Tuesday 31 January 2017 14:21
- Target OS: Raspberry Pi / ODroid
- Domoticz version: V3.6328
- Location: Sollentuna, Sweden
- Contact:
Re: A few "cosmetic bugs" in the user interface
I said I would look into it, so I finally took me a little time tonight. Here is a reasonably simple solution that I came up with, to enable the translation of the "Last Seen:" label.
EDIT: I also took the opportunity to resolve the same issue with the "Type:" labels, as that string was also defined in the CSS. It should be noted that both of the translated strings already exist in the localized language files. But they were never actually used before, since there was no mechanism for handling localization of strings defined in the CSS files!
First, I commented out ALL of the sections (there are 8 or 9 of these) in the 'style.css' that were setting the "content" and "font-style" properties on the "Last Seen" fiels. Additionally, I did the same for ALL the sections (I think there were 3) targeting the "Type" fields. Below are examples of such sections:
Then I introduced the following lines at the very beginning of a js that was loaded when a page was initialized. For my little Proof-Of-Concept here, I did it in 'DashboardController.js', but maybe it should go into one of the more generic JavaScripts instead, to assure that it will be used on all pages:
Just a short note: Over the years, I have written in more than 20 different programming/scripting languages, but javascript is definitely NOT one of my favorites, so the code might appear a bit primitive. Especially the awkward way I pad on the required quotes to the local variable strings, but it works. Please see it mainly as a Proof-Of-Concept and feel free to do whatever tweaking you deem necessary!
And please also be aware that I have not verified if these modifications possibly might have some other unforeseen implications!
/P-G
EDIT: I also took the opportunity to resolve the same issue with the "Type:" labels, as that string was also defined in the CSS. It should be noted that both of the translated strings already exist in the localized language files. But they were never actually used before, since there was no mechanism for handling localization of strings defined in the CSS files!
First, I commented out ALL of the sections (there are 8 or 9 of these) in the 'style.css' that were setting the "content" and "font-style" properties on the "Last Seen" fiels. Additionally, I did the same for ALL the sections (I think there were 3) targeting the "Type" fields. Below are examples of such sections:
Code: Select all
/* body table#itemtable tr td:first-child + td + td + td + td:before{
content: "Last Seen: ";
font-style: italic;
} */
/* body table#itemtable tr td:first-child + td + td + td + td:before{
content: "Type: ";
font-style: italic;
} */
Code: Select all
var langLastSeen='"' + ($.t("Last Seen")) + ': ";';
var langTypeText='"' + ($.t("Type")) + ': ";';
document.styleSheets[0].insertRule('td#lastupdate::before { content: ' + langLastSeen + '; font-style: italic;}', 0);
document.styleSheets[0].insertRule('td#type::before { content: ' + langTypeText + '; font-style: italic;}', 0);
And please also be aware that I have not verified if these modifications possibly might have some other unforeseen implications!
/P-G
- PeGe
- Posts: 25
- Joined: Tuesday 31 January 2017 14:21
- Target OS: Raspberry Pi / ODroid
- Domoticz version: V3.6328
- Location: Sollentuna, Sweden
- Contact:
Re: A few "cosmetic bugs" in the user interface
I did some further testing this morning, and it seems to work fine. I have not seen any negative consequences so far.
A few screenshots of various languages below:
A few screenshots of various languages below:
- Attachments
-
- Switch_de.png (20.27 KiB) Viewed 2844 times
-
- Switch_se.png (19.92 KiB) Viewed 2844 times
-
- Switch_en.png (19.08 KiB) Viewed 2844 times
-
- Posts: 107
- Joined: Tuesday 31 January 2017 21:54
- Target OS: Raspberry Pi / ODroid
- Domoticz version:
- Contact:
Re: A few "cosmetic bugs" in the user interface
if you can modify dark theme with those i could happily test
Who is online
Users browsing this forum: No registered users and 1 guest