> On Aug. 23, 2015, 2:47 a.m., Jeremy Whiting wrote: > > Ship It! > > Jeremy Whiting wrote: > Incidentally, the pt issue sounds like a QLocale bug, do we need to find > one or file a new one for that probably?
Yeah, looks a Qt bug, it has a table from CLDR of default countries to use so there's something wrong with the lookup. I'm not aware of a bug report for that so go ahead. That, or Thiago has rigged it so he always gets pt_BR :-) - John ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124885/#review84202 ----------------------------------------------------------- On Aug. 23, 2015, 1:34 a.m., Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124885/ > ----------------------------------------------------------- > > (Updated Aug. 23, 2015, 1:34 a.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
