[Konsole-devel] Review Request: Open links from konsole with single mouse click

[Jekyll Wu]

> On July 12, 2012, 2:35 p.m., Jekyll Wu wrote:
> > I think it is OK to go except some small issues. 
> > 
> > If you already have a developer account, please fix those issues and commit it to the master branch (remember use the "REVIEW:<id>" hook in the commit message to automatically close this review ).  If not, I will do it for you.
> >
> 
> Asko Eronen wrote:
>     I'm afraid I don't have a developer account.

OK, then I will do it.


- Jekyll


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


On July 10, 2012, 6:23 p.m., Asko Eronen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105282/
> -----------------------------------------------------------
> 
> (Updated July 10, 2012, 6:23 p.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> Added optional feature in konsole, which allows opening links with single mouse click.
> 
> By default, the option is disabled. It can be enabled from profile manager -> Mouse -> "Mouse Click opens links"
> 
> 
> Diffs
> -----
> 
>   src/EditProfileDialog.h f9e35e6 
>   src/EditProfileDialog.cpp 678636e 
>   src/EditProfileDialog.ui e4196de 
>   src/Profile.h 10a71b6 
>   src/Profile.cpp aded720 
>   src/TerminalDisplay.h 8d91ddb 
>   src/TerminalDisplay.cpp 6a47299 
>   src/ViewManager.cpp 0d01aae 
> 
> Diff: http://git.reviewboard.kde.org/r/105282/diff/
> 
> 
> Testing
> -------
> 
> Tested with simple terminal usage using gentoo package manager, irssi and htop. No problems ocured with them.
> 
> 
> Thanks,
> 
> Asko Eronen
> 
>

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