Bug 406973 - Not possible to translate the ‘Reasons’ strings (for saving into non-Krita formats)
Summary: Not possible to translate the ‘Reasons’ strings (for saving into non-Krita fo...
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Translation (show other bugs)
Version: git master (please specify the git hash!)
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Karl Ove Hufthammer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-04-27 14:24 UTC by Karl Ove Hufthammer
Modified: 2019-08-15 07:53 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Ove Hufthammer 2019-04-27 14:24:41 UTC
SUMMARY
It’s not possible to translate the three ‘Reason:/‘Reasons:’ strings, shown in the warning messages you get when trying to save an image into a non-Krita format.

ADDITIONAL INFORMATION
Here’s the relevant strings in the source code (KisImportExportManager.cpp, https://lxr.kde.org/source/extragear/graphics/krita/libs/ui/KisImportExportManager.cpp), AFAICS:

0482     if (!batchMode && !errors.isEmpty()) {
0483         QString error =  "<html><body><p><b>"
0484                 + i18n("Error: cannot save this image as a %1.", mimeUserDescription)
0485                 + "</b> Reasons:</p>"
0486                 + "<p/><ul>";
0487         Q_FOREACH(const QString &w, errors) {
0488             error += "\n<li>" + w + "</li>";
0489         }

0526             QString warning = "<html><body><p><b>"
0527                     + i18n("You will lose information when saving this image as a %1.", mimeUserDescription);
0528 
0529             if (warnings.size() == 1) {
0530                 warning += "</b> Reason:</p>";
0531             }
0532             else {
0533                 warning += "</b> Reasons:</p>";
0534             }
0535             warning += "<p/><ul>";
Comment 1 Tiar 2019-04-27 20:51:31 UTC
Hey!

Basically, error messages in Krita are a one big mess. I'm working on changing them to be more unified.

Please go to https://invent.kde.org/kde/krita/merge_requests/15 to see my sggestion of new error messages (that would replace the old ones, whatever the old ones were; and they were a lot of things...). There is a string freeze before May 1st, so I need to get it merged to master before that.

Please comment on the merge request if you have any suggestions.
Comment 2 Halla Rempt 2019-04-29 13:24:00 UTC
This is completely different from the error messages you get when saving, though: these are the warnings that are shown beforehand, and those warnings aren't a mess, but rather well designed and implemented :P
Comment 3 Halla Rempt 2019-04-29 13:25:04 UTC
No, I have to take that back... This isn't about that part of the code, sorry for being so confused.
Comment 4 Karl Ove Hufthammer 2019-08-14 19:31:03 UTC
Git commit c091fce255d42f45e6add84451a64a5349941bd0 by Karl Ove Hufthammer.
Committed on 14/08/2019 at 19:30.
Pushed by huftis into branch 'master'.

Make string for reasons for image export problems translatable

M  +3    -3    libs/ui/KisImportExportManager.cpp

https://invent.kde.org/kde/krita/commit/c091fce255d42f45e6add84451a64a5349941bd0
Comment 5 Halla Rempt 2019-08-15 07:53:50 UTC
Git commit 3dd45fdb1066042c59aa09cf014050a441c8ba47 by Boudewijn Rempt, on behalf of Karl Ove Hufthammer.
Committed on 15/08/2019 at 07:46.
Pushed by rempt into branch 'krita/4.2'.

Make string for reasons for image export problems translatable

M  +3    -3    libs/ui/KisImportExportManager.cpp

https://invent.kde.org/kde/krita/commit/3dd45fdb1066042c59aa09cf014050a441c8ba47