all repos — fluxbox @ 4663a9591e08b554f82366cdd8e9075b0a6fbe55

custom fork of the fluxbox windowmanager 

bugfix: 'HideMenus' command did not work if the user 'opened' an empty submenu.
'openeing' such an empty submenu lead to an already hidden menu referenced by
the 'shown' pointer
Mathias Gumz akira at fluxbox dot org
commit

4663a9591e08b554f82366cdd8e9075b0a6fbe55

parent

cfd055064a6b02f05aa6b77d7d26a9b33ad8a1eb

2 files changed, 6 insertions(+), 1 deletions(-)

jump to
M ChangeLog → ChangeLog 
@@ -1,5 +1,10 @@ 
  (Format: Year/Month/Day)
 Changes for 1.1.2
+
+*09/09/23:
+   * Bugfix: Empty submenus lead to misfunction of the 'HideMenus' command (Mathias)
+     FbTk/Menu.cc
+
 *09/09/12:
    * Implemented visibility of titlebar buttons via the apps file (Mathias)
      Window.cc
M src/FbTk/Menu.cc → src/FbTk/Menu.cc 
@@ -759,7 +759,7 @@ item->submenu()->move(new_x, new_y);
         if (! m_moving)
             clearItem(index);
 
-        if (! item->submenu()->isVisible()) {
+        if (! item->submenu()->isVisible() && item->submenu()->numberOfItems() > 0) {
             shown = item->submenu();
             item->showSubmenu();
             item->submenu()->raise();