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

René J.V. Bertin rjvbertin at gmail.com
Fri Sep 4 15:40:25 BST 2015



> 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 for OSX I suggest to create a branch for your patches.

As I wrote in the summary, I don't consider this so much a new feature as a fix to an omission because the parameter is used in kdelibs (and possible elsewhere I don't know about). Besides, this concerns a KCM that I think should have been part of kde-runtime (but that's probably a moot point).

Also, this is not only about OS X. There are several distribution releases that ship KDE4 as the default desktop officially supported LTS version, and I'd hope they too would be interested in upstream fixes. As such I don't see the point in creating another branch, or in maintaining a freeze on a branch that isn't going to see any more releases
A separate repository with only fixes, organised by project and possibly target platform could make sense though.


- René J.V.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125043/#review84821
-----------------------------------------------------------


On Sept. 4, 2015, 4:22 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> 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 Mac OS X, kde-workspace and kdelibs.
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> KDE4 has been providing a setting that (would have) allowed to avoid unwanted text zooming during simulated inertial scrolling (scroll coasting). KDE PIM applications were immune to that issue because certain KDELibs classes use the parameter, which made it all the more annoying that other (e.g. Kate-based) applications weren't. Sadly this setting wasn't published via a GUI.
> 
> This patch adds a checkbox to the input ("mouse") KCM which seemed like the most appropriate place if not only because it also makes sense to provide this KCM on non-X11 platforms like OS X and MS Windows (where settings like "double or single click" are relevant).
> 
> I consider this a fix of an omission bug, but I realise that it could also be considered a new feature, so this RR is also intended to give some public exposure to my patch rather than keeping it to myself.
> 
> 
> Diffs
> -----
> 
>   kcontrol/input/kmousedlg.ui b48a606 
>   kcontrol/input/mouse.h d926a99 
>   kcontrol/input/mouse.cpp cebb174 
> 
> Diff: https://git.reviewboard.kde.org/r/125043/diff/
> 
> 
> Testing
> -------
> 
> For now only on OS X with kdelibs 4.14.11 .
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20150904/f6508c8e/attachment.htm>


More information about the kde-core-devel mailing list