[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #18 from Tyson Tan --- Created attachment 139758 --> https://bugs.kde.org/attachment.cgi?id=139758=edit Krita Layers Docker root name not translated The "root"prefix of a merged layer is not translated. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #17 from Tyson Tan --- Created attachment 139757 --> https://bugs.kde.org/attachment.cgi?id=139757=edit Krita Layers Docker default Background Layer name string has no translation -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #16 from Tyson Tan --- Hi Tiar, I can confirm the default palette is now loading and the translated names of palettes and gradients are now showing in the tooltips. Thanks! For the untranslated things I noticed: Brushe presets has no translation, krita.pot don't have entries for them. I think it's very important to make them translatable. Are you still planning to have translation for them in the future? The "Background" name string for default Background layer in Layers docker does not show translation. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #15 from Tiar --- Hi @Tyson, the current state of affairs: 1) The resource name shown as the name is non-translated. 2) The translated resource name is shown on the tooltip. (Note that (1) and (2) are consistent with the design document: "We currently translate brush presets and brush tips and other Gimp-based resources. We will stop doing this, because it makes it very difficult to support tutorials. However, we will show the translations as tooltips." I am thinking that maybe it should show the translated names, but that decision needs to be made by Halla). 3) Hence, the Default palette is found, because it looks for the palette by name, which is not translated. 4) Tags are translatable and translated to Polish, at least. Seem to be working fine. (They have "url" which is not translated and "name" which is translated and shown to the user). 5) To see my fixes, you need to recreate the database, because the old is wrong (for example there are translated names for palette etc.). 6) More resource has translated resource names than before (because I added proper context when pulling the string from the po file). For example, all palettes are translated now, it's just to see the translation, you need to hover over the palette in the palette chooser... 7) To get resources and tags in a different language, you still need to remove the old database. This bug was reported here: bug 439300 to not make too much of a mess here. 8) If you find *any* resource that you know that you translated but it's not translated in Krita, please tell me (with the resource name, the language the translation is in etc.). -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #14 from Tiar --- Git commit 5074a4327a8222e448a31282f75a6722689fe6fd by Agata Cacko. Committed on 29/06/2021 at 12:47. Pushed by tymond into branch 'master'. Add translated name to the tooltip Before this commit, there was no way to see the translated resource name, because the Qt::DisplayRole shows the name, instead of the 'tooltip' (translated name). This commit adds the translated name to the tooltip (only if the translated name is different from the main name). M +7-1libs/resourcewidgets/KisIconToolTip.cpp https://invent.kde.org/graphics/krita/commit/5074a4327a8222e448a31282f75a6722689fe6fd -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #13 from Tiar --- Git commit 9c685a3f40205f0637c1e13d2fe88fab01c4eefa by Agata Cacko. Committed on 29/06/2021 at 12:47. Pushed by tymond into branch 'master'. Remove unnecessary hack in Favourite Resource Manager Before this commit, Favourite Resource Manager tried to find "My Favourites" tag in the tag model by looking for the star symbol. This is unnecessary because earlier in the code it tries to find the correct tag by url. Urls are not translatable and cannot be changed, so it should be enough to just look for the url, no matter the language. M +0-15 libs/ui/kis_favorite_resource_manager.cpp https://invent.kde.org/graphics/krita/commit/9c685a3f40205f0637c1e13d2fe88fab01c4eefa -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #12 from Tiar --- Git commit 788a8a3d9ff8b222771e6e4967bd44029839a79e by Agata Cacko. Committed on 29/06/2021 at 12:47. Pushed by tymond into branch 'master'. Use correct context for translated resource names Before this commit, CacheDb wouldn't use any context, which meant that translated resource names wouldn't be used (because the translated names had a context). This commit provides that context so a lot more resource names are translated. Note: I can't say for sure if the context is correct for all resources. If you find a resource that should have been translated in the database but isn't, please reopen the bug report. M +2-1libs/resources/KisResourceCacheDb.cpp https://invent.kde.org/graphics/krita/commit/788a8a3d9ff8b222771e6e4967bd44029839a79e -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #11 from Tiar --- Git commit 72b1ba46fa118bec6979dfbca4d8e3e525f091fa by Agata Cacko. Committed on 29/06/2021 at 12:47. Pushed by tymond into branch 'master'. Fix finding default palette in layersplit Before this commit, Layersplit would try to find the palette with a translated name "Default". In krita 5, we don't have translated names, so it wouldn't be able to find it. This commit changes it so it looks for the untranslated name. M +1-1plugins/extensions/layersplit/dlg_layersplit.cpp https://invent.kde.org/graphics/krita/commit/72b1ba46fa118bec6979dfbca4d8e3e525f091fa -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 Tiar changed: What|Removed |Added Latest Commit||https://invent.kde.org/grap ||hics/krita/commit/c018c6c31 ||27b7adfb8f0e738f3c957f19ba0 ||dfd2 Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --- Comment #10 from Tiar --- Git commit c018c6c3127b7adfb8f0e738f3c957f19ba0dfd2 by Agata Cacko. Committed on 29/06/2021 at 12:47. Pushed by tymond into branch 'master'. Fix palette name being translated Before this commit, palette name was translated, which meant the default palette wasn't found. Translations of the resource names should be stored in `tooltip` instead. This commit removes translation from the resource name, but the tooltip is still translated, as it should be. (Tested with, I believe, Japanese language). M +2-1libs/pigment/resources/KoColorSet.cpp https://invent.kde.org/graphics/krita/commit/c018c6c3127b7adfb8f0e738f3c957f19ba0dfd2 -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #9 from Tiar --- Note about the recent commit if anyone is confused: it was added as "related" (CCBUG) to this bug report because while it's in a different component and about a tag, not a resource, it seems like code-wise it was caused by the same thing (forgetting about translation of the name). I added a hack, true fix would change the strings... @Tyson I just learned that apparently *tooltips* are supposed to hold translated resources names. I would need to talk about it more with Halla. However, she's on vacations... I will look into the mystery of why only some resources have translated names. --- Is that a regression (aka does it not happen in Krita 4.4.5)? -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #8 from Tyson Tan --- Thank you Tiar. I can confirm the tags are translated when 5.0 starts from a clean installation. However, they do not become translated when upgrading from 4.4.5 to 5.0. The only resource names that have translatable strings are: - Palette (default, gradient) - Gradient (FG to BG, FG to Transparent) Those names have been existed and translated at least since 3 years ago. However, for some reason, only the ones in () are currently used. No other resource name has a translatable strings. I thought Krita 5.0 would allow us to have translated brushes and such. Wasn't it the case? -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #7 from Tiar --- Git commit c75e336c15b4855c38f7a67ebdab67ba5d56926d by Agata Cacko. Committed on 25/06/2021 at 21:25. Pushed by tymond into branch 'master'. Find default tag for popup palette Before this commit, the code would use an old method to get the tag it wanted, and it wouldn't have a way to get the default tag (My Favourities) for non-English systems. Because user strings cannot be changed for now, this commit includes a hack that tries to find first tag with star in in, assuming it's the Favourities tag. M +25 -7libs/ui/kis_favorite_resource_manager.cpp https://invent.kde.org/graphics/krita/commit/c75e336c15b4855c38f7a67ebdab67ba5d56926d -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #6 from Tiar --- @Tyson, currently the way to get translated tags is to make sure that when the database is created, the Krita's language matches the language you want your tags in. Can you please confirm on your system that that's the case? And possibly, but not necessarily, for the resources too? PS. If I copy and paste some of the translatable strings from other parts of the code, does it break the string freeze, or can I do it safely? -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 --- Comment #5 from Tyson Tan --- Hi Tiar, In Split Layer dialogue, the "Default" palette is selected by default, and its name has been translated. However, for some reason only "Default" and "Gradient" in the palette list have translated names. I have already translated all the names in krita.pot (/krita/data/palettes/). None of the resource names have been translated. Not even the tags. I recognized some strings in krita._desktop.pot looks like they are supposed to be tags, but they aren't used in the app. There aren't any other resource strings available for translate -- brushes, patterns, none. -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470
--- Comment #4 from Tiar ---
Can you please confirm that in Layer -> Split -> Split Layer, there is a
selected palette and it's the "Default" palette?
It seems like fortunately someone thought about translation in that one place
so I can fix it before Krita 5. by using `i18n("Default")` since it's available
in
plugins/extensions/layersplit/dlg_layersplit.cpp:55:QString paletteName =
cfg.readEntry("layersplit/paletteName", i18n("Default"));
For others, I'll need to check but are the resource names translated? Like
brush presets? I know that tags are translated.
--
You are receiving this mail because:
You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 Tiar changed: What|Removed |Added Status|REPORTED|ASSIGNED Ever confirmed|0 |1 Assignee|krita-bugs-n...@kde.org |tamtamy.tym...@gmail.com CC||tamtamy.tym...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[krita] [Bug 438470] Color Palette Docker loads nothing by default on non-English locales
https://bugs.kde.org/show_bug.cgi?id=438470 Tyson Tan changed: What|Removed |Added Summary|Color Palette Docker loads |Color Palette Docker loads |nothing by default on |nothing by default on |non-English locale |non-English locales -- You are receiving this mail because: You are watching all bug changes.
