----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118716/#review59949 -----------------------------------------------------------
+9000 for the ever present comments! kcms/translations/kcmtranslations.cpp <https://git.reviewboard.kde.org/r/118716/#comment41729> Why are you having John's name translated? ;) kcms/translations/kcmtranslations.cpp <https://git.reviewboard.kde.org/r/118716/#comment41730> New signal-slots syntax? kcms/translations/kcmtranslations.cpp <https://git.reviewboard.kde.org/r/118716/#comment41727> remove (then you can also remove the qdebug include) kcms/translations/kcmtranslations.cpp <https://git.reviewboard.kde.org/r/118716/#comment41728> Why are there <p> tags? - Martin Klapetek On June 13, 2014, 4: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, 4: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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel