[Kde-games-devel] Review Request 110650: Enable ctrl-m (by default) to toggle the menubar, with a prompt.

[Matthew Scheirer]

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

(Updated June 1, 2013, 5:43 p.m.)


Review request for KDE Games.


Changes
-------

Menubar isn't in an consistent state until after the call to setupGUI() in the window constructor, so I moved the check to the end of it. You were right, menubar() reported false no matter what until the GUI state is loaded.


Description
-------

Adds the ctrl-m standard keyboard shortcut to hide the menubar, with a warning prompt. Mainly for my grandfather who would right click the menubar and diable the main toolbar and not know what to do.


This addresses bug 320269.
    http://bugs.kde.org/show_bug.cgi?id=320269


Diffs (updated)
-----

  mainwindow.h c9cbeae 
  mainwindow.cpp a42f8ef 

Diff: http://git.reviewboard.kde.org/r/110650/diff/


Testing
-------

Shortcut works, settings preserved across multiple sessions.


Thanks,

Matthew Scheirer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20130601/b55cace1/attachment-0001.html>