[Konsole-devel] Review Request: Add 'Ctrl+=' as the alternate shortcut for enlarging font size

[Commit Hook]

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


This review has been submitted with commit 4de54208d095641a67a245604c64d3c31b65060c by Jekyll Wu to branch master.

- Commit Hook


On Jan. 9, 2012, 7:31 p.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103661/
> -----------------------------------------------------------
> 
> (Updated Jan. 9, 2012, 7:31 p.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> 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.
> 
> 
> Diffs
> -----
> 
>   src/SessionController.cpp 60d3f36 
> 
> Diff: http://git.reviewboard.kde.org/r/103661/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

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