Review Request 118716: New Translations KCM
Sebastian Kügler
sebas at kde.org
Fri Jun 13 12:06:57 UTC 2014
> On June 13, 2014, 9:59 a.m., Martin Gräßlin wrote:
> > kcms/translations/kcmtranslations.cpp, lines 54-55
> > <https://git.reviewboard.kde.org/r/118716/diff/1/?file=280673#file280673line54>
> >
> > this is a library. One may NEVER use setApplicationDomain in a library, it would break the translation of systemsettings, see http://api.kde.org/frameworks-api/frameworks5-apidocs/ki18n/html/classKLocalizedString.html#ad866b11bf396b9d93b7dc313a1930b7b
>
> Sebastian Kügler wrote:
> Hm, then we might need this one, after all:
>
> https://git.reviewboard.kde.org/r/118678/
>
> Martin, do you think this would be the correct solution?
>
> Martin Gräßlin wrote:
> sorry, I don't get what you want. You want to translate the KCM, don't you? You have add_definitions(-DTRANSLATION_DOMAIN=\"kcmtranslations\") in your CMakeLists.txt, nothing else is needed.
>
> Sebastian Kügler wrote:
> I want to list translations for a given domain, which might, or might not be the current one. This is about querying KLS, not about translations themselves.
>
> Martin Gräßlin wrote:
> ok, then I don't know the solution, but calling ::setApplicationDomain is wrong.
Well, I proposed a solution, and asked if you think that's a correct one: https://git.reviewboard.kde.org/r/118678/ :)
This patch basically doesn't touch applicationDomain in KLS, just allows querying for any domain.
- Sebastian
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118716/#review59954
-----------------------------------------------------------
On June 13, 2014, 2:31 a.m., Sebastian Kügler wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118716/
> -----------------------------------------------------------
>
> (Updated June 13, 2014, 2:31 a.m.)
>
>
> Review request for KDE Frameworks, Plasma, Chusslove Illich, and John Layt.
>
>
> Repository: plasma-desktop
>
>
> Description
> -------
>
> This is the second part of the replacement of the Locale KCM: Translations.
>
> This KCM complements the formats KCM, which sets the locale. With this KCM, the user can configure GUI translations.
>
> The mechanism to set up the langhere, KF5-uage is the same as in the formats KCM, while formats set LC_* and LANG, this KCM will set LANGUAGE. The UI is roughly the same as in the previous "Locale", though here, KF5-compatible mechanism (QLocale, ki18n, are used.
>
> This code has been written by John Layt, I've merely cleaned it up a bit, added some missing features, tested it and am submitting it now for review.
>
>
> Diffs
> -----
>
> kcms/CMakeLists.txt 6bda5434d7d1147c0e8295f2919c17ea59d1be99
> kcms/formats/kcmformats.h 345fbb193d327c39fd80d690bdf8f41d3171c6a2
> kcms/formats/kcmformats.cpp 4169244f68b4f3c766cae514a8dde4d902cbfee1
> kcms/formats/writeexports.h PRE-CREATION
> kcms/translations/CMakeLists.txt dd70f47d11e2f3a1359cae2a1306b217af82d848
> kcms/translations/Messages.sh 38f48cb32af5ff278bb25b6bef323c546e9223a3
> kcms/translations/kcmtranslations.h adc3a48d1b65cbba89d74326eb42c7925a3708b2
> kcms/translations/kcmtranslations.cpp 9095fda40ed5c794f06c56ed842bf7861dc88f2e
> kcms/translations/kcmtranslationswidget.ui 4ad3242811d3f92acc05d010e120189f681446ad
> kcms/translations/translations.desktop ea9086732f7359ba58496d8b60851eca34669c16
>
> Diff: https://git.reviewboard.kde.org/r/118716/diff/
>
>
> Testing
> -------
>
> Set up languages (x-test, others), re-logged in, languages correctly set up for KF5 and console applications.
>
>
> File Attachments
> ----------------
>
> KCM in systemsettings
> https://git.reviewboard.kde.org/media/uploaded/files/2014/06/13/28a00558-f671-4f39-bfe3-ca9dca8e98ce__tranlationskcm.png
>
>
> Thanks,
>
> Sebastian Kügler
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140613/f425ff0b/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list