all repos — fluxbox @ 6febc40c47f8acc2c5afb78990d794303ab61b14

custom fork of the fluxbox windowmanager 

bugfix: place WindowMenu correctly upon first call (closes #2731524)

FbTk::Menu::updateMenu() recalculates the width() and height() of the
window. this must be done before the menu is moved to fit completely
onto the screen.

the 'menu().raise()' command is not needed, that is done inside
'menu().show()' already.
Mathias Gumz akira at fluxbox dot org
commit

6febc40c47f8acc2c5afb78990d794303ab61b14

parent

9616a676d869b8f67742da99d55e9327ea867166

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

jump to
M src/Window.cc → src/Window.cc 
@@ -1887,6 +1887,8 @@ menu().reloadHelper()->checkReload();
 
     int head = screen().getHead(menu_x, menu_y);
 
+    menu().updateMenu(); // recalculate the menu size
+
     // move menu directly under titlebar but not off the screen
     if (menu_y < static_cast<signed>(screen().maxTop(head)))
         menu_y = screen().maxTop(head);

@@ -1901,7 +1903,6 @@ 
     FbMenu::setWindow(this);
     menu().move(menu_x, menu_y);
     menu().show();
-    menu().raise();
     menu().grabInputFocus();
 }