Bug 463976 - [kcm_users] Text context not suitable for translation
Summary: [kcm_users] Text context not suitable for translation
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: general (other bugs)
Version First Reported In: master
Platform: Other Other
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-01-07 20:12 UTC by Emir SARI
Modified: 2023-01-11 16:36 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Emir SARI 2023-01-07 20:12:09 UTC
SUMMARY

The following string is not possible to translate properly in Turkish:

"Please repeatedly %1 your %2 on the fingerprint sensor."

In the notes section, it is indicated that %1 is a verb (e.g. scan), and %2 is the name of a finger. For Turkish, due to the differences in verb conjugation, formal approach and vowel harmony, it is not possible to simply replace a place holder with a verb. Unlike English, it has to be conjugated due to the agglutinative nature of the Turkish language.

Let's go with the "scan" verb. The translation for "to scan" is "taramak", but I need to conjugate it as "tara-yın" to direct the action to the user. For another possible verb, the suffix might be "-yun", "-in", or something else because vowel harmony requires me to adopt a new suffix according to the verb root. So I am unable to blindly put a suffix for all.

For the finger name, let's say "index finger". It is "işaret parmağı", but this one also requires conjugation as well (işaret parmağınızı). The conjugation here does not violate the vowel harmony, but it is really not good practice to blindly put placeholders like this. I don't even know where the finger names are defined.

The ideal solution would be including all the possible action verbs and finger names and their all possible context exclusively in this file, so at least we can have an idea of how to modify them as needed.
Comment 1 Bug Janitor Service 2023-01-08 17:21:04 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/2488
Comment 2 David Edmundson 2023-01-10 18:11:23 UTC
Git commit 1068f3e0dc2b0d97eb8fcf4fa690aad2738d5200 by David Edmundson, on behalf of Nicolas Fella.
Committed on 10/01/2023 at 16:40.
Pushed by davidedmundson into branch 'master'.

[kcms/users] Fix i18n for finger enrollment messages

We can't concatenate sentences like this, it's impossible to properly translate.

Instead we have to spell out all 2*10 combinations of scantype and finger

M  +50   -3    kcms/users/package/contents/ui/FingerprintDialog.qml
M  +3    -2    kcms/users/src/fingerprintmodel.cpp
M  +2    -2    kcms/users/src/fingerprintmodel.h
M  +16   -3    kcms/users/src/fprintdevice.cpp
M  +7    -1    kcms/users/src/fprintdevice.h
M  +1    -0    kcms/users/src/kcm.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/1068f3e0dc2b0d97eb8fcf4fa690aad2738d5200
Comment 3 Emir SARI 2023-01-11 16:36:29 UTC
Thank you very much!