----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127800/#review95056 -----------------------------------------------------------
To have localized numbers is the expected behavior (and documented as such). But it was "temporarily" disabled between kdelibs and KF5, until replacements for KLocale were decided. I don't remember why I didn't switch to %L1 at that moment, since it was available too... Well, I guess best to commit this and see later if there is any trouble. Did you run unit tests? Could be some of them needs adapting. - Chusslove Illich On Мај 1, 2016, 12:47 пре п., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127800/ > ----------------------------------------------------------- > > (Updated Мај 1, 2016, 12:47 пре п.) > > > Review request for KDE Frameworks. > > > Repository: ki18n > > > Description > ------- > > Use %L1 instead of %1 which tells it it's a number. > > This way I get decimal indicator and thousands separators according to my > locale in eg. KUnitConversion > > > Diffs > ----- > > src/klocalizedstring.cpp fc80135 > > Diff: https://git.reviewboard.kde.org/r/127800/diff/ > > > Testing > ------- > > 0,00094697205065430554 Meilen and 1.524 Millimeter > instead of > 0.00094697205065430554 Meilen and 1524 Millimeter > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
