> On Aug. 22, 2015, 7:47 p.m., Jeremy Whiting wrote: > > Ship It!
Incidentally, the pt issue sounds like a QLocale bug, do we need to find one or file a new one for that probably? - Jeremy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124885/#review84202 ----------------------------------------------------------- On Aug. 22, 2015, 6:34 p.m., Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124885/ > ----------------------------------------------------------- > > (Updated Aug. 22, 2015, 6:34 p.m.) > > > Review request for Plasma. > > > Bugs: 345761 and 347956 > https://bugs.kde.org/show_bug.cgi?id=345761 > https://bugs.kde.org/show_bug.cgi?id=347956 > > > Repository: plasma-desktop > > > Description > ------- > > Fixes two problems: > * Variants not being shown up, i.e. ca ca@valencia showing up both as > "català" > * pt showing up as "português do Brasil" > > For the first one i've went the easy route of adding the languageCode if > there's an @ in it > For pt i hard to hard code it since i found no other way to make Qt > understand that for "pt" we mean portuguese from portugal > > > Diffs > ----- > > kcms/translations/kcmtranslations.cpp e5024e2 > > Diff: https://git.reviewboard.kde.org/r/124885/diff/ > > > Testing > ------- > > > Thanks, > > Albert Astals Cid > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
