[Tellico-users] Re: Review Request: Add shortcut key to show/hide menubar (Bug 251157).

[Robby Stephenson]

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

Ship it!


- Robby


On May 13, 2011, 5:43 p.m., Pedro MC wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101357/
> -----------------------------------------------------------
> 
> (Updated May 13, 2011, 5:43 p.m.)
> 
> 
> Review request for Tellico.
> 
> 
> Summary
> -------
> 
> Add shortcut key to show/hide menu bar. No menu entry is added.
> One thing to be noted is that I changed the shortcut key for the internet search to CTRL+I, from CTRL+M, to conform with the standard show/hide menu shortcut key that is CTRL+M.
> 
> 
> This addresses bug 251157.
>     http://bugs.kde.org/show_bug.cgi?id=251157
> 
> 
> Diffs
> -----
> 
>   ChangeLog 60c09c3 
>   src/mainwindow.h 88c1f16 
>   src/mainwindow.cpp c425415 
> 
> Diff: http://git.reviewboard.kde.org/r/101357/diff
> 
> 
> Testing
> -------
> 
> Tested feature and it worked as expected.
> 
> 
> Thanks,
> 
> Pedro
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/tellico-users/attachments/20110514/2c29b7a0/attachment.html>