<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/103661/">http://git.reviewboard.kde.org/r/103661/</a>
     </td>
    </tr>
   </table>
   <br />



 <p>Ship it!</p>



 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I vaguely recall a bug report asking for this, but I couldn't find it.  Please commit it.  Thanks.</pre>
 <br />







<p>- Kurt</p>


<br />
<p>On January 9th, 2012, 7:31 p.m., Jekyll Wu 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 Jekyll Wu.</div>


<p style="color: grey;"><i>Updated Jan. 9, 2012, 7:31 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;">Due to the way how key '-' and key '+' are positioned in most keyboards, users need to press three keys to enlarge font size, instead of the intuitive two keys just like the shrinking case.  

AFAIK, most web brosers support "Ctrl + =" in addition to "Ctrl + +" for zooming. Okular and Gwenview also provide that convenience by default. So I would suggest to add this convenience into konsole.

pros: it makes users life a bit easier by default

cons: it makes konsole intercept another keystroke combination by default. Currently, 'Paste' is the only action which has alternate shortcut by default.

</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/SessionController.cpp <span style="color: grey">(60d3f36)</span></li>

</ul>

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




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








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