| Summary: | too long translation in Plasma Battery applet | ||
|---|---|---|---|
| Product: | [Translations] i18n | Reporter: | Rolf Eike Beer <kde> |
| Component: | de | Assignee: | German-Translators <kde-i18n-de> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | nate, plasma-bugs, schwarzer |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Attachments: | screenshot | ||
|
Description
Rolf Eike Beer
2021-02-03 13:19:48 UTC
Created attachment 135392 [details]
screenshot
Sorry, but this is not a translation issue but a layouting issue. https://techbase.kde.org/Development/Tutorials/Localization/i18n_Mistakes#Pitfall_.231:_Pixel_Based_Layouts It's not a layout issue; we are indeed using layouts, and there is plenty of space here for a shorter translation, but the applet size is fixed. Therefore it is important that short strings and short translations be used in plasma applets. FWIW there is more horizontal space available in Plasma 5.21 because we moved the buttons to the header. So it may already be accidentally fixed there. But still, a shorter translation would be better if possible. Moving to the i18n product to see if that's possible. Correct me if I am wrong. Depending on the system font configured by the user the maximum length for that string varies. ... If that is the case, it is not possible to find a properly shortened string that will work. In this special case the label could be broken into two lines if too long. The size of the applet's popup adjusts with the font metrics and size. So roughly, the amount of space for text should scale. Since this text is in a checkbox, there is a technical issue that prevents it from being able to have two lines. So we're stuck with either making more space, or shortening the translation. SVN commit 1592508 by schwarzer: shorten translation a bit to fit it into fixed width layout M +2 -2 plasma_applet_org.kde.plasma.battery.po WebSVN link: http://websvn.kde.org/?view=rev&revision=1592508 I hope this fits now. Eike, could you please check, if this is working now? Yes, short enough. Thx. |