Summary: | Several i18n and RTL-layout issues | ||
---|---|---|---|
Product: | [Applications] digikam | Reporter: | Safa Alfulaij <safa1996alfulaij> |
Component: | Usability-i18n | Assignee: | Digikam Developers <digikam-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | aspotashev, caulier.gilles, metzpinguin, sh.yaron |
Priority: | NOR | Keywords: | rtl |
Version: | 7.5.0 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/graphics/digikam/commit/81da1575e1407698e5cd4d28b7c444627877fb6d | Version Fixed In: | 7.5.0 |
Description
Safa Alfulaij
2017-02-13 11:03:05 UTC
Git commit 0b47db718258add4058584cbeb056bea3d0aadf6 by Gilles Caulier. Committed on 13/02/2017 at 19:55. Pushed by cgilles into branch 'master'. point 1/ fixed M +34 -31 utilities/assistants/firstrun/welcomepage.cpp M +1 -1 utilities/assistants/firstrun/welcomepage.h https://commits.kde.org/digikam/0b47db718258add4058584cbeb056bea3d0aadf6 Links are dead. New links! 2. https://invent.kde.org/graphics/digikam/-/blob/master/core/utilities/geolocation/geoiface/correlator/gpscorrelatorwidget.cpp#L517 "%1 out of %2 images have been correlated." 3. https://invent.kde.org/graphics/digikam/-/blob/master/core/utilities/geolocation/geoiface/reversegeocoding/rgwidget.cpp#L971 "%1 out of %2 images have been reverse geocoded. " 4. https://invent.kde.org/graphics/digikam/-/blob/master/core/data/about/main.html 5. https://invent.kde.org/graphics/digikam/-/blob/master/core/app/main/digikamapp.cpp#L614-647 All strings needs plural, and the order of placeholders needs to be fixed Safa, Hi and happy new year Please take attention to this bugzilla entry where we apply plenty of fixes to improve i18n in digiKam : https://bugs.kde.org/show_bug.cgi?id=438701 And please, make a try of digiKam 7.5.0 pre-release installer for Windows available here: https://files.kde.org/digikam/ A feedback to see if RTL support in digiKam 7.5.0 have been improved will be appreciate. Thanks in advance Gilles Caulier Point 2/ : comment from source code : "no need for i18np here, because the case of correlationTotalCount == 1 is covered in the other two cases." ==> No fix is required... Git commit d65063ef33134f90895fc0ddafd686f4b531551e by Gilles Caulier. Committed on 31/12/2021 at 15:02. Pushed by cgilles into branch 'master'. Fix point 3/ about non plurial case M +15 -4 core/utilities/geolocation/geoiface/reversegeocoding/rgwidget.cpp https://invent.kde.org/graphics/digikam/commit/d65063ef33134f90895fc0ddafd686f4b531551e Point 4: the Welcome page image properties strings are non translatable because it's hard coded in HTML code. Solution : port the welcome page view to pure Qt code, not using QtWebkit/QtWebEngine. Advantage : this will drop one part of digiKam code depending of QtWebKit or QtWebEngine ==> maintenance of code will be more easy. Maik, If I'm not too wrong, point 5/ do not require plural handling in source code as a top level condition only call this code for more than one item. Right ? Gilles Hi Gilles, Yes, I agree with you, at least 2 must be selected for this text to be displayed, item(s) is correct here. Maik Ok. So : Point 5/ : INVALID. Gilles Hi all. Happy new year! Thanks for checking this. Regarding points 2 and 5, I'd reference this section of the tutorial here: https://techbase.kde.org/Development/Tutorials/Localization/i18n#Plurals Even if the string is displayed for 1+ numbers, and English and many European countries doesn't have more than 1 plural form for that case (that's the "many" form), many other languages have different forms to say 2 images or 14 images or 23 images, etc. You can check the list of language plural forms here: http://docs.translatehouse.org/projects/localization-guide/en/latest/l10n/pluralforms.html And check the languages that have more than 2 conditions. Thanks! Safa, Do you mean that we needs for these cases to use plural forms in source code ? Best Gilles Yes, the points I raised needs to be plural, 2 and 5 included. Git commit 1ce54dec5db0612dbfe99f4f375e910a1d9ce49a by Gilles Caulier. Committed on 09/01/2022 at 10:01. Pushed by cgilles into branch 'master'. add missing i18n from Welcome page view, Background image section M +7 -1 core/app/views/stack/welcomepageview.cpp M +5 -5 core/data/about/main.html https://invent.kde.org/graphics/digikam/commit/1ce54dec5db0612dbfe99f4f375e910a1d9ce49a 4. https://cgit.kde.org/digikam.git/tree/data/about/main.html#n55 The picture metadata are not translatable. ==> fixed with my last commit. Git commit ed900e592e1582984cf8c39f87891176850627bc by Gilles Caulier. Committed on 09/01/2022 at 10:13. Pushed by cgilles into branch 'master'. Fix i18np (point #2) M +12 -5 core/utilities/geolocation/geoiface/correlator/gpscorrelatorwidget.cpp https://invent.kde.org/graphics/digikam/commit/ed900e592e1582984cf8c39f87891176850627bc Git commit 5ed1ca11742f357f411ec93a2ccacb7b7fbb44ed by Gilles Caulier. Committed on 09/01/2022 at 10:26. Pushed by cgilles into branch 'master'. fix i18np (point #5) M +46 -34 core/app/main/digikamapp.cpp https://invent.kde.org/graphics/digikam/commit/5ed1ca11742f357f411ec93a2ccacb7b7fbb44ed All i18n point are now fixed in source code. Still RTL layout issues, point 1/ to 4/ Gilles Caulier Git commit cfe78b294de58667f70c6e0b8d10259d36be2ed7 by Gilles Caulier. Committed on 09/01/2022 at 10:51. Pushed by cgilles into branch 'master'. Fix RTL support (point #1) M +3 -1 core/app/views/stack/welcomepageview.cpp M +14 -14 core/data/about/main.html https://invent.kde.org/graphics/digikam/commit/cfe78b294de58667f70c6e0b8d10259d36be2ed7 Git commit dcd896822efe8da630533e80f13f3bfb0340e177 by Gilles Caulier. Committed on 09/01/2022 at 12:40. Pushed by cgilles into branch 'master'. fix LTR (point #4) M +6 -11 core/libs/dialogs/dsplashscreen.cpp M +0 -1 core/libs/dialogs/dsplashscreen.h https://invent.kde.org/graphics/digikam/commit/dcd896822efe8da630533e80f13f3bfb0340e177 Git commit ebc2a0b6acfdde98ab114839f6da5f58b363cb2c by Gilles Caulier. Committed on 10/01/2022 at 09:21. Pushed by cgilles into branch 'master'. fix LTR (point #3) M +9 -1 core/libs/properties/itempropertiestab.cpp M +22 -3 core/libs/properties/itempropertiestxtlabel.cpp M +9 -1 core/utilities/searchwindow/choicesearchutilities.cpp M +20 -2 core/utilities/searchwindow/searchtabheader.cpp https://invent.kde.org/graphics/digikam/commit/ebc2a0b6acfdde98ab114839f6da5f58b363cb2c Git commit 81da1575e1407698e5cd4d28b7c444627877fb6d by Gilles Caulier. Committed on 10/01/2022 at 09:53. Pushed by cgilles into branch 'master'. fix LTR (point #2) FIXED-IN: 7.5.0 M +46 -10 core/libs/widgets/itemview/ditemtooltip.cpp https://invent.kde.org/graphics/digikam/commit/81da1575e1407698e5cd4d28b7c444627877fb6d Git commit a2c94122536bc1636742cd2cedf2bc24fc1ee5b7 by Gilles Caulier. Committed on 13/01/2022 at 04:26. Pushed by cgilles into branch 'master'. Fix i18np args verified with krazy M +1 -1 core/utilities/geolocation/geoiface/correlator/gpscorrelatorwidget.cpp https://invent.kde.org/graphics/digikam/commit/a2c94122536bc1636742cd2cedf2bc24fc1ee5b7 Git commit bf0e0a23212e182f8e8ab111398994977327fe01 by Gilles Caulier. Committed on 13/01/2022 at 04:30. Pushed by cgilles into branch 'master'. Fix i18np args verified with krazy M +5 -5 core/app/main/digikamapp.cpp https://invent.kde.org/graphics/digikam/commit/bf0e0a23212e182f8e8ab111398994977327fe01 |