> On ago. 24, 2015, 8:16 a.m., David Faure wrote: > > Minor suggestions, feel free to ignore if no time. > > > > I wonder though, Qt returning ca-ES from uiLanguages and ca_ES from > > QLocale, wouldn't that be a bug in Qt?
It's not a bug per se but just Qt being not very smart and using names in iso in one place and in bcp47 in other, and not really documenting it either. - Albert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124884/#review84251 ----------------------------------------------------------- On ago. 24, 2015, 10:10 p.m., Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124884/ > ----------------------------------------------------------- > > (Updated ago. 24, 2015, 10:10 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kxmlgui > > > Description > ------- > > QLocale and us don't always agree on how to call things, for example we call > our translation "ca" while QLocale gives us "ca_ES" in locale::name so we > need to strip the _ in some cases. > > To make things funnier, QLocale::uiLanguages actually returns ca-ES so we > first turn all the - to _ > > > Diffs > ----- > > src/kswitchlanguagedialog_p.cpp 2ae4b0b > > Diff: https://git.reviewboard.kde.org/r/124884/diff/ > > > Testing > ------- > > tested rsibreak, works + See screenshots > > > File Attachments > ---------------- > > Before > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/22/79c70322-b55d-4940-a167-b98309eaa6ec__before.png > After > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/22/f3eae50b-53a1-4836-809d-ede574c80690__after.png > > > Thanks, > > Albert Astals Cid > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
