Re: Review Request 118347: [klipper] Port from KIntSpinBox to QSpinBox
> On May 29, 2014, 2:30 p.m., David Edmundson wrote: > > >Unfortunately this way we cannot have a plural aware suffix. > > > > Then why port away? > > Linking against kde4libsupport seems like the lesser of two evils. > > Aleix Pol Gonzalez wrote: > Well then we need to decide how we're going to get to the proper fix. We > can't just decide it's good enough to depend on kdelibs4. > > David Edmundson wrote: > We also can't just break things. > > Options are: > 1) restore KIntSpinBox to kwidgetaddons > 2) fix Qt, and drop the lib dependency once we have 5.4 > > I was just looking at Qt translator code. Detecting if something is > singular or plural is not an easy task, and you need to have the locale tell > you. There's not a nice system that fits in with Qt's API. I guess we can > file a bug report anyway and see what happens. > > Qt bug opened https://bugreports.qt-project.org/browse/QTBUG-39356 - David --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118347/#review58724 --- On May 27, 2014, 12:04 p.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118347/ > --- > > (Updated May 27, 2014, 12:04 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > --- > > [klipper] Port from KIntSpinBox to QSpinBox > > Unfortunately this way we cannot have a plural aware suffix. > > > Diffs > - > > klipper/configdialog.cpp 15901992db8d6fe28d800c4a18a3a6529cace1be > klipper/generalconfig.ui f0145752eb2dbd927392b682b6472695c793a588 > > Diff: https://git.reviewboard.kde.org/r/118347/diff/ > > > Testing > --- > > > Thanks, > > Martin Gräßlin > > ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 118347: [klipper] Port from KIntSpinBox to QSpinBox
> On May 29, 2014, 2:30 p.m., David Edmundson wrote: > > >Unfortunately this way we cannot have a plural aware suffix. > > > > Then why port away? > > Linking against kde4libsupport seems like the lesser of two evils. > > Aleix Pol Gonzalez wrote: > Well then we need to decide how we're going to get to the proper fix. We > can't just decide it's good enough to depend on kdelibs4. We also can't just break things. Options are: 1) restore KIntSpinBox to kwidgetaddons 2) fix Qt, and drop the lib dependency once we have 5.4 I was just looking at Qt translator code. Detecting if something is singular or plural is not an easy task, and you need to have the locale tell you. There's not a nice system that fits in with Qt's API. I guess we can file a bug report anyway and see what happens. - David --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118347/#review58724 --- On May 27, 2014, 12:04 p.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118347/ > --- > > (Updated May 27, 2014, 12:04 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > --- > > [klipper] Port from KIntSpinBox to QSpinBox > > Unfortunately this way we cannot have a plural aware suffix. > > > Diffs > - > > klipper/configdialog.cpp 15901992db8d6fe28d800c4a18a3a6529cace1be > klipper/generalconfig.ui f0145752eb2dbd927392b682b6472695c793a588 > > Diff: https://git.reviewboard.kde.org/r/118347/diff/ > > > Testing > --- > > > Thanks, > > Martin Gräßlin > > ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 118347: [klipper] Port from KIntSpinBox to QSpinBox
> On May 29, 2014, 2:30 p.m., David Edmundson wrote: > > >Unfortunately this way we cannot have a plural aware suffix. > > > > Then why port away? > > Linking against kde4libsupport seems like the lesser of two evils. Well then we need to decide how we're going to get to the proper fix. We can't just decide it's good enough to depend on kdelibs4. - Aleix --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118347/#review58724 --- On May 27, 2014, 12:04 p.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118347/ > --- > > (Updated May 27, 2014, 12:04 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > --- > > [klipper] Port from KIntSpinBox to QSpinBox > > Unfortunately this way we cannot have a plural aware suffix. > > > Diffs > - > > klipper/configdialog.cpp 15901992db8d6fe28d800c4a18a3a6529cace1be > klipper/generalconfig.ui f0145752eb2dbd927392b682b6472695c793a588 > > Diff: https://git.reviewboard.kde.org/r/118347/diff/ > > > Testing > --- > > > Thanks, > > Martin Gräßlin > > ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 118347: [klipper] Port from KIntSpinBox to QSpinBox
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118347/#review58724 --- >Unfortunately this way we cannot have a plural aware suffix. Then why port away? Linking against kde4libsupport seems like the lesser of two evils. - David Edmundson On May 27, 2014, 12:04 p.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118347/ > --- > > (Updated May 27, 2014, 12:04 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > --- > > [klipper] Port from KIntSpinBox to QSpinBox > > Unfortunately this way we cannot have a plural aware suffix. > > > Diffs > - > > klipper/configdialog.cpp 15901992db8d6fe28d800c4a18a3a6529cace1be > klipper/generalconfig.ui f0145752eb2dbd927392b682b6472695c793a588 > > Diff: https://git.reviewboard.kde.org/r/118347/diff/ > > > Testing > --- > > > Thanks, > > Martin Gräßlin > > ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 118347: [klipper] Port from KIntSpinBox to QSpinBox
> On May 27, 2014, 3:52 p.m., Martin Klapetek wrote: > > > Unfortunately this way we cannot have a plural aware suffix. > > > > What if you do this from code like "setSuffix(i18np("signular", "plural", > > value));" ? > > Martin Klapetek wrote: > Oh but that's not a binding so it won't auto update...too much time in > QML xD > > Nevermind. I hoped there is a better solution, which I'm just not aware of :-) It would need a connect to update the suffix whenever the value changes, which was the reason I didn't implement that one. - Martin --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118347/#review58567 --- On May 27, 2014, 2:04 p.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118347/ > --- > > (Updated May 27, 2014, 2:04 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > --- > > [klipper] Port from KIntSpinBox to QSpinBox > > Unfortunately this way we cannot have a plural aware suffix. > > > Diffs > - > > klipper/configdialog.cpp 15901992db8d6fe28d800c4a18a3a6529cace1be > klipper/generalconfig.ui f0145752eb2dbd927392b682b6472695c793a588 > > Diff: https://git.reviewboard.kde.org/r/118347/diff/ > > > Testing > --- > > > Thanks, > > Martin Gräßlin > > ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 118347: [klipper] Port from KIntSpinBox to QSpinBox
> On May 27, 2014, 3:52 p.m., Martin Klapetek wrote: > > > Unfortunately this way we cannot have a plural aware suffix. > > > > What if you do this from code like "setSuffix(i18np("signular", "plural", > > value));" ? Oh but that's not a binding so it won't auto update...too much time in QML xD Nevermind. - Martin --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118347/#review58567 --- On May 27, 2014, 2:04 p.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118347/ > --- > > (Updated May 27, 2014, 2:04 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > --- > > [klipper] Port from KIntSpinBox to QSpinBox > > Unfortunately this way we cannot have a plural aware suffix. > > > Diffs > - > > klipper/configdialog.cpp 15901992db8d6fe28d800c4a18a3a6529cace1be > klipper/generalconfig.ui f0145752eb2dbd927392b682b6472695c793a588 > > Diff: https://git.reviewboard.kde.org/r/118347/diff/ > > > Testing > --- > > > Thanks, > > Martin Gräßlin > > ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Re: Review Request 118347: [klipper] Port from KIntSpinBox to QSpinBox
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118347/#review58567 --- > Unfortunately this way we cannot have a plural aware suffix. What if you do this from code like "setSuffix(i18np("signular", "plural", value));" ? - Martin Klapetek On May 27, 2014, 2:04 p.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118347/ > --- > > (Updated May 27, 2014, 2:04 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > --- > > [klipper] Port from KIntSpinBox to QSpinBox > > Unfortunately this way we cannot have a plural aware suffix. > > > Diffs > - > > klipper/configdialog.cpp 15901992db8d6fe28d800c4a18a3a6529cace1be > klipper/generalconfig.ui f0145752eb2dbd927392b682b6472695c793a588 > > Diff: https://git.reviewboard.kde.org/r/118347/diff/ > > > Testing > --- > > > Thanks, > > Martin Gräßlin > > ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
Review Request 118347: [klipper] Port from KIntSpinBox to QSpinBox
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118347/ --- Review request for Plasma. Repository: plasma-workspace Description --- [klipper] Port from KIntSpinBox to QSpinBox Unfortunately this way we cannot have a plural aware suffix. Diffs - klipper/configdialog.cpp 15901992db8d6fe28d800c4a18a3a6529cace1be klipper/generalconfig.ui f0145752eb2dbd927392b682b6472695c793a588 Diff: https://git.reviewboard.kde.org/r/118347/diff/ Testing --- Thanks, Martin Gräßlin ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel