Bug 74795 - PATCH: When menubar is visible, change menu text to "Hide Menubar"
Summary: PATCH: When menubar is visible, change menu text to "Hide Menubar"
Status: RESOLVED FIXED
Alias: None
Product: konsole
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Konsole Developer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-02-10 07:59 UTC by Kurt Hindenburg
Modified: 2004-04-26 23:37 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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()