[Konsole-devel] Review Request: Add setting for konsole control drag

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105423/#review15478
-----------------------------------------------------------


This review has been submitted with commit ed31a8b0aa6fcd45177202a60db7167e2498c80f by Kurt Hindenburg to branch master.

- Commit Hook


On July 5, 2012, 4:25 p.m., Scott Kitterman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105423/
> -----------------------------------------------------------
> 
> (Updated July 5, 2012, 4:25 p.m.)
> 
> 
> Review request for Konsole and Kurt Hindenburg.
> 
> 
> Description
> -------
> 
>     Description: Add setting for control drag
>     Both KDE 3 and KDE 4.7 and 4.8 default to requiring click and control to drag
>     test in konsole.  This was not required in KDE 4.0 - 4.6 and some people
>     prefer each method.  At the moment, changing this requires recompiling
>     konsole, which is suboptimal.  This patch adds a selection option in
>     the profile configuration options to change this at run time.
>     Addresses complaints about the fix in https://bugs.kde.org/show_bug.cgi?id=165923.
> 
> 
> This addresses bug 165923.
>     http://bugs.kde.org/show_bug.cgi?id=165923
> 
> 
> Diffs
> -----
> 
>   src/EditProfileDialog.h 2345925 
>   src/EditProfileDialog.cpp f5cfa8c 
>   src/EditProfileDialog.ui 8b8e83c 
>   src/Profile.h 83616c4 
>   src/Profile.cpp 67cd80e 
>   src/TerminalDisplay.h c6e5972 
>   src/TerminalDisplay.cpp 112925f 
>   src/ViewManager.cpp 92ab0b1 
> 
> Diff: http://git.reviewboard.kde.org/r/105423/diff/
> 
> 
> Testing
> -------
> 
> The most recent KDE I have running is 4.8.4.  I first developed this patch for 4.8.4, compiled it and verified click/drag worked either with or without control based on the setting.  The code required minimal adaptation to bring it up to trunk.
> 
> 
> Screenshots
> -----------
> 
> Designer Screenshot with master
>   http://git.reviewboard.kde.org/r/105423/s/613/
> Revised screenshot from designer
>   http://git.reviewboard.kde.org/r/105423/s/617/
> 
> 
> Thanks,
> 
> Scott Kitterman
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20120706/d66cf694/attachment.html>