<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/105282/">http://git.reviewboard.kde.org/r/105282/</a>
     </td>
    </tr>
   </table>
   <br />





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi,

First of all, thank you for your patch and the time you've put into it.
Then to drop a little bomb. I strongly suggest _not_ committing this. KDE has a click policy for single or double click. It's hidden in the system settings somewhere. I think it's a better idea to let that policy work with Konsole rather then making a Konsole only option for single/double click.

Sorry for the little bomb.

Cheers,
Mark</pre>
 <br />







<p>- Mark</p>


<br />
<p>On June 17th, 2012, 7:22 p.m., Asko Eronen wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Konsole.</div>
<div>By Asko Eronen.</div>


<p style="color: grey;"><i>Updated June 17, 2012, 7:22 p.m.</i></p>






<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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"</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Tested with simple terminal usage using gentoo package manager, irssi and htop. No problems ocured with them.</pre>
  </td>
 </tr>
</table>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>src/EditProfileDialog.h <span style="color: grey">(2345925e3998f92c1d1fcaae85c36c30782c319f)</span></li>

 <li>src/EditProfileDialog.cpp <span style="color: grey">(f5cfa8c4a715683f31da93183d69979e7cec8818)</span></li>

 <li>src/EditProfileDialog.ui <span style="color: grey">(8b8e83cd5a31ea2a4fd7dbdfcfa1db0b9af5adba)</span></li>

 <li>src/Profile.h <span style="color: grey">(83616c4fe62dd1c3bc70d12b62ed0d876d5f0fce)</span></li>

 <li>src/Profile.cpp <span style="color: grey">(67cd80ea84af47c446071cbcc05af6af4d049733)</span></li>

 <li>src/TerminalDisplay.h <span style="color: grey">(c6e5972d5bfa9dd21bc0d880a56a76efece578f7)</span></li>

 <li>src/TerminalDisplay.cpp <span style="color: grey">(849ce0d0598a2d6a1085bd8776882dcfd66d21ea)</span></li>

 <li>src/ViewManager.cpp <span style="color: grey">(40ed9150cec61dc8677839d3c5e47ea7b22b65fd)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/105282/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>