Bug 74795

Summary: PATCH: When menubar is visible, change menu text to "Hide Menubar"
Product: [Applications] konsole Reporter: Kurt Hindenburg <khindenburg>
Component: generalAssignee: Konsole Developer <konsole-devel>
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Kurt Hindenburg 2004-02-10 07:59:37 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources
Compiler:          gcc version 3.2.3 20030422 (Gentoo Linux 1.4 3.2.3-r3, propolice) 
OS:          Linux

This uses the new setCheckedState in KDE libs.

When menubar is visible, text is "Hide Menubar".  When menubar is hidden, text is "Show Menubar".

One liner:
--- konsole.cpp.orig    2004-02-10 01:46:34.252558584 -0500
+++ konsole.cpp 2004-02-10 01:51:01.448938560 -0500
@@ -419,6 +419,7 @@
       {
          showMenubar->plug ( m_rightButton, index );
          m_rightButton->insertSeparator( index+1 );
+         showMenubar->setCheckedState( KGuiItem( "Hide &Menubar", "showmenu", QString::null, QString::null ) );
       }
       index = 2;
       m_rightButton->setItemVisible(POPUP_NEW_SESSION_ID,true);
Comment 1 Stephan Binner 2004-02-10 08:13:48 UTC
It's still an ToggleAction. Why make it inconsistent to every other Show Menubar/Toolbar KDE-wide?
Comment 2 Kurt Hindenburg 2004-02-10 17:06:05 UTC
I dislike the current appearance (not just konsole).  The checked/unchecked icon appearance is confusing.  And the text says "Show" even when the menubar is shown. This is just my opinion of course.
I like the appearance with this patch; much more obvious to the normal user.
Comment 3 Stephan Binner 2004-04-26 23:37:29 UTC
Committed, but in makeBasicGUI()