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

[Albert Astals Cid]

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


I see this has a ship it but marked as uncommited. Was it commited and you forgot to close the request or is still uncommited?

- Albert Astals Cid


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.
> 
> 
> Description
> -------
> 
> 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 MC
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/tellico-users/attachments/20130207/4195f322/attachment.html>