Re: Fullscreen mode

[Abdelrazak Younes]

Pavel Sanda wrote:
You could also introduce LFUN_STATUSBAR_TOGGLE by the way if you want to 
achieve real full screen.

does the following patch looks more sensible to you?

Yes but I guess it could be a bit shorter:

@@ -1524,6 +1525,15 @@ bool GuiView::dispatch(FuncRequest const & cmd)
                        }
                        break;
                }
+               case LFUN_MENUBAR_TOGGLE:
+                   menuBar()->setVisible(!menuBar()->isVisible());
+                   break;
+
+               case LFUN_STATUSBAR_TOGGLE:
+                   statusBar()->setVisible(!statusBar()->isVisible());
+                   break;
+

Then you don't need the additional toggling methods. By the way you 
don't need to enclose the cases between accolades if there is no 
declaration within it.

Abdel.