Review Request 124884: Fix KSwitchLangaugeDialog

[Albert Astals Cid]

> 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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150824/9e65aebe/attachment-0001.html>