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.

1 files changed, 2 insertions, 1 deletions

diff --git a/src/Window.cc b/src/Window.cc
index 79e9b43..c87180d 100644
--- a/src/Window.cc
+++ b/src/Window.cc
@@ -1887,6 +1887,8 @@ void FluxboxWindow::showMenu(int menu_x, int menu_y) {
 
     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 @@ void FluxboxWindow::showMenu(int menu_x, int menu_y) {
     FbMenu::setWindow(this);
     menu().move(menu_x, menu_y);
     menu().show();
-    menu().raise();
     menu().grabInputFocus();
 }