Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125043/ --- Review request for KDE Software on Mac OS X, kde-workspace and kdelibs.

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125043/#review84816 --- kcontrol/input/mouse.cpp (lines 183 - 185)

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125043/#review84819 --- kcontrol/input/kmousedlg.ui (line 82)

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread René J . V . Bertin
> On Sept. 4, 2015, 3:21 p.m., Thomas Lübking wrote: > > kcontrol/input/mouse.cpp, line 184 > > > > > > read this line closely, then fix it ;-P Oops :) But I guess that answers my question whether the change

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread René J . V . Bertin
> On Sept. 4, 2015, 3:38 p.m., Martin Gräßlin wrote: > > You are aware that this is a dead repo and that this is a new feature for a > > repository that has been feature frozen for years? > > > > Given that I think this should not and never be merged. If you want to keep > > the repo going

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125043/#review84822 --- hmm, looks like a feature in a now frozen repo, not sure is a

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125043/ --- (Updated Sept. 4, 2015, 4:22 p.m.) Review request for KDE Software on

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread Luigi Toscano
> On Set. 4, 2015, 3:38 p.m., Martin Gräßlin wrote: > > You are aware that this is a dead repo and that this is a new feature for a > > repository that has been feature frozen for years? > > > > Given that I think this should not and never be merged. If you want to keep > > the repo going for

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread Luigi Toscano
> On Set. 4, 2015, 3:21 p.m., Thomas Lübking wrote: > > kcontrol/input/kmousedlg.ui, line 82 > > > > > > You *must* ask the i18n team about this (requires an exception) on > > kde-...@kde.org > > > >

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125043/#review84821 --- You are aware that this is a dead repo and that this is a new

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread Luigi Toscano
> On Set. 4, 2015, 3:38 p.m., Martin Gräßlin wrote: > > You are aware that this is a dead repo and that this is a new feature for a > > repository that has been feature frozen for years? > > > > Given that I think this should not and never be merged. If you want to keep > > the repo going for

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread Jeremy Whiting
> On Sept. 4, 2015, 7:38 a.m., Martin Gräßlin wrote: > > You are aware that this is a dead repo and that this is a new feature for a > > repository that has been feature frozen for years? > > > > Given that I think this should not and never be merged. If you want to keep > > the repo going

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread René J . V . Bertin
> On Sept. 4, 2015, 3:38 p.m., Martin Gräßlin wrote: > > You are aware that this is a dead repo and that this is a new feature for a > > repository that has been feature frozen for years? > > > > Given that I think this should not and never be merged. If you want to keep > > the repo going

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread René J . V . Bertin
> On Sept. 4, 2015, 3:38 p.m., Martin Gräßlin wrote: > > You are aware that this is a dead repo and that this is a new feature for a > > repository that has been feature frozen for years? > > > > Given that I think this should not and never be merged. If you want to keep > > the repo going

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread Martin Klapetek
> On Sept. 4, 2015, 3:38 p.m., Martin Gräßlin wrote: > > You are aware that this is a dead repo and that this is a new feature for a > > repository that has been feature frozen for years? > > > > Given that I think this should not and never be merged. If you want to keep > > the repo going

Re: Review Request 125043: expose the WheelMouseZooms global setting through the input ("mouse") KCM

2015-09-04 Thread René J . V . Bertin
> On Sept. 4, 2015, 3:38 p.m., Martin Gräßlin wrote: > > You are aware that this is a dead repo and that this is a new feature for a > > repository that has been feature frozen for years? > > > > Given that I think this should not and never be merged. If you want to keep > > the repo going