<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 />





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 4de54208d095641a67a245604c64d3c31b65060c by Jekyll Wu to branch master.</pre>
 <br />







<p>- Commit</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>