Bug 399440 - RTL aligned text and Hebrew text when application language is English
Summary: RTL aligned text and Hebrew text when application language is English
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Translation (show other bugs)
Version: 4.1.3
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords: rtl
Depends on:
Blocks:
 
Reported: 2018-10-06 12:37 UTC by Tamir Bahar
Modified: 2022-09-20 10:18 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Hebrew text and some alignment issues (6.20 KB, image/png)
2018-10-06 12:37 UTC, Tamir Bahar
Details
Label placement issues (6.94 KB, image/png)
2018-10-06 12:38 UTC, Tamir Bahar
Details
Misaligned menu text and parentheses (18.48 KB, image/png)
2018-10-06 12:38 UTC, Tamir Bahar
Details
Properly aligned parens (21.92 KB, image/png)
2020-06-28 12:39 UTC, Tamir Bahar
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tamir Bahar 2018-10-06 12:37:31 UTC
Created attachment 115444 [details]
Hebrew text and some alignment issues

Some parts of the Krita interface are badly alligned (RTL) or showing Hebrew text despite the application language being set to English.

STEPS TO REPRODUCE
I have Windows 10 x64.
The language is English.
The locale is Hebrew (I tried changing it, it did not affect Krita)
Location is Israel
Language list includes Hebrew and English.

I have not managed to pinpoint the system-setting that causes Krita to act this way. I'll be happy to provide any additional information if I know what to look for.

OBSERVED RESULT
As visible in the attached screenshots - the menus look a bit out-of-sorts. With alignment issues & some Hebrew text.

EXPECTED RESULT
The menus should appear identical to those visible on English-only systems.
Comment 1 Tamir Bahar 2018-10-06 12:38:06 UTC
Created attachment 115445 [details]
Label placement issues
Comment 2 Tamir Bahar 2018-10-06 12:38:30 UTC
Created attachment 115446 [details]
Misaligned menu text and parentheses
Comment 3 Halla Rempt 2018-10-08 08:44:37 UTC
Git commit b68419b0395002d9043da6bacc2a7510bbce3844 by Boudewijn Rempt.
Committed on 08/10/2018 at 08:43.
Pushed by rempt into branch 'master'.

Check whether the override language is RTL or LTR

If we have an override language, Qt doesn't accept that, even though
we set the entire environment correctly. This means we get translated
dialog box buttons, but we cannot do anything about that. This attempts
to at least fix the application layout.

M  +25   -10   krita/main.cc

https://commits.kde.org/krita/b68419b0395002d9043da6bacc2a7510bbce3844
Comment 4 Halla Rempt 2018-10-08 08:45:52 UTC
Hi Tamir,

Could you please check with the next nightly windows build (#314) to see whether my hack fixes the layout issues for you? I cannot test myself, because if I set my desktop to hebrew or any other RTL language, I won't be able to set it back to something I can read...

https://binary-factory.kde.org/job/Krita_Nightly_Windows_Build/
Comment 5 Tamir Bahar 2018-10-08 19:23:22 UTC
I'll check the nightly build when it completes.

As for the translated buttons - there is an easy workaround for that. Removing the `he` directory from `Krita (x64)/bin/locale` prevents the translation.
Comment 6 Halla Rempt 2018-10-09 07:08:55 UTC
If I understand the Qt code, that should even prevent the RTL/LTR confusion, and if it doesn't, Qt is even weirder than I thought :-)
Comment 7 Tamir Bahar 2018-10-09 16:21:47 UTC
I checked with the #314 nightly (https://binary-factory.kde.org/job/Krita_Nightly_Windows_Build/314/). There is no change.
The menus are misaligned and the Hebrew is showing.
Removing the `he` locale directory only removes the translation, and does not fix the alignment.
Comment 8 Halla Rempt 2018-10-09 16:45:41 UTC
Dash it... Maybe I should add a manual override option then.
Comment 9 Safa Alfulaij 2020-06-18 15:34:17 UTC
Hi Tamir
Is this still an issue? If you're still facing it please provide more information. I might test as well (using a RTL locale, Arabic)
Comment 10 Tamir Bahar 2020-06-27 08:43:18 UTC
Looks like it's mostly resolved now. Some placements are still iffy, but I think it's mostly due to lack of translations.
Comment 11 Safa Alfulaij 2020-06-27 09:28:32 UTC
Can you show what you mean by screenshots?
Comment 12 Tamir Bahar 2020-06-28 12:39:31 UTC
Created attachment 129741 [details]
Properly aligned parens

You can see that the parentheses are now properly aligned.
The rest looks the same.
The questions is - where a translation is missing for a specific part - should it be flipped for RTL, or kept as LTR?
Comment 13 Justin Zobel 2020-11-13 03:09:53 UTC
Removing assigned tag as commit hasn't fixed the issue.
Comment 14 Halla Rempt 2020-11-13 08:35:45 UTC
Do not change the status of bugs, Justin.
Comment 15 Alvin Wong 2021-12-30 11:07:59 UTC
Please recheck with 5.0, I think it should be fixed?
Comment 16 Halla Rempt 2022-09-14 08:31:06 UTC
Setting to needsinfo, since it needs someone with the appropriate locale to check with the latest release.
Comment 17 Tamir Bahar 2022-09-14 14:35:01 UTC
Sorry for the long delay on my part.

I tested now (Krita 5.1.1) and it seems to work as expected!
Comment 18 Halla Rempt 2022-09-14 14:35:31 UTC
Yay, thanks for testing!!!
Comment 19 Bug Janitor Service 2022-09-17 04:36:11 UTC
Thanks for your comment!

Automatically switching the status of this bug to REPORTED so that the KDE team
knows that the bug is ready to get confirmed.

In the future you may also do this yourself when providing needed information.