commit 7b060d1c2acf03378c87abb7215f9c0a957b549c
author Rafael Lima <rafael.palma.lima@gmail.com> Sun Jul 23 14:57:58 2023 +0200
committer Michael Weghorn <m.weghorn@posteo.de> Mon Jul 24 17:27:12 2023 +0200
tree 48d6a5273fd83fa0956d1b2354d1412f31b7019e
parent 2b2fdfa0e9aea207a065506c56c2a0033b4cbd44

tdf#154447 Fix menu position in kf5

Prior to this patch the menus were using "topLeft" as reference for positioning the menu, which made them appear on top of the control that originated their appearance.

With this patch, it now uses "bototmLeft" to place the menu below the control (similar to gtk3).

Change-Id: I66b312fe09e078ec067773a483f9e59f627fcf93
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154771
Tested-by: Jenkins
Tested-by: Gabor Kelemen <kelemeng@ubuntu.com>
Reviewed-by: Gabor Kelemen <kelemeng@ubuntu.com>
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>

vcl/qt5/QtMenu.cxx

diff --git a/vcl/qt5/QtMenu.cxx b/vcl/qt5/QtMenu.cxx
index dca0305..b239b45 100644
--- a/vcl/qt5/QtMenu.cxx
+++ b/vcl/qt5/QtMenu.cxx
@@ -841,7 +841,7 @@ bool QtMenu::ShowNativePopupMenu(FloatingWindow* pWin, const tools::Rectangle& r
    assert(pFrame);
    const tools::Rectangle aFloatRect = FloatingWindow::ImplConvertToAbsPos(xParent, rRect);
    const QRect aRect = toQRect(aFloatRect, 1 / pFrame->devicePixelRatioF());
    mpQMenu->exec(aRect.topLeft());
    mpQMenu->exec(aRect.bottomLeft());

    return true;
}