On 20/02/2014 00:10, Pavel Sanda wrote:
Pavel Sanda wrote:
Hi,

it seems we forbid ui-toggle statusbar in fullscreen mode for no good reason.

Please can someone on windows and mac test for me that the following patch
does not break fullscreen statusbar absence?
Hey, last appeal before commiting :)

I guess this looks reasonable. I can't really see a reason at this point why it was like this, sorry.

Abdel.



For testing simply switch to fullscreen and try to use menu via shortcut
and observe whether it looks as expected, namely statusbar does not appear
or flicker on the screen.

It should not, but I have very little idea what would be another reason to
include these two lines of code otherwise (Abdel?).

Pavel
diff --git a/src/frontends/qt4/GuiView.cpp b/src/frontends/qt4/GuiView.cpp
index 006c7e8..8acc94a 100644
--- a/src/frontends/qt4/GuiView.cpp
+++ b/src/frontends/qt4/GuiView.cpp
@@ -3801,8 +3801,6 @@ void GuiView::dispatch(FuncRequest const & cmd, 
DispatchResult & dr)
        if (isFullScreen()) {
                if (menuBar()->isVisible() && lyxrc.full_screen_menubar)
                        menuBar()->hide();
-               if (statusBar()->isVisible())
-                       statusBar()->hide();
        }
  }

Reply via email to