[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    [PATCH] #129464 Konqueror hiding/showing menubar in fullscreen mode
From:       Tadeusz Andrzej Kadlubowski <yess () kolor ! univ ! gda ! pl>
Date:       2006-07-22 11:30:52
Message-ID: 20060722113051.GA25759 () hell ! org ! pl
[Download RAW message or body]

Hello,

This is my first KDE patch. :)

It fixes konqueror. The "show/hide menubar" options can be reversed in some
circumstances when in fullscreen mode.

It does not fix similar problem when quitting konqueror with menubar hidden and
restarting it. I will investigate this issue later.

Please apply a patch that is attached to this mail. It works for both 3.5
branch and for the trunk. It applies to the file
kdebase/konqueror/konq_mainwindow.cc.

Regards,
-- 
tadeusz andrzej kadłubowski

["konq_mainwindow.cc.patch" (text/plain)]

--- konq_mainwindow.cc.orig	2006-07-22 12:30:55.000000000 +0200
+++ kdebase/konqueror/konq_mainwindow.cc	2006-07-22 11:53:05.000000000 +0200
@@ -3545,7 +3545,8 @@
 
 
     menuBar()->hide();
-
+    m_paShowMenuBar->setChecked( false );
+ 
     // Qt bug, the flags are lost. They know about it.
     // happens only with the hackish non-_NET_WM_STATE_FULLSCREEN way
     setWFlags( WDestructiveClose );
@@ -3570,6 +3571,7 @@
     unplugActionList( "fullscreen" );
 
     menuBar()->show(); // maybe we should store this setting instead of forcing it
+    m_paShowMenuBar->setChecked( true );
 
     // Qt bug, the flags aren't restored. They know about it.
     setWFlags( WType_TopLevel | WDestructiveClose );


>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic