Bug 156483

Summary: main menu pop-up isn't on its normal place
Product: LibreOffice Reporter: Roman Kuznetsov <79045_79045>
Component: UIAssignee: Not Assigned <libreoffice-bugs>
Status: RESOLVED DUPLICATE    
Severity: normal CC: 79045_79045, miguelangelrv, m.weghorn, telesto
Priority: medium Keywords: bibisectRequest, regression
Version: 24.2.0.0 alpha0+   
Hardware: All   
OS: All   
Whiteboard:
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 85811    

Description Roman Kuznetsov 2023-07-26 20:05:22 UTC 
Description:
main menu pop-up isn't on its normal place

Steps to Reproduce:
1. Open current LibreOffice TDF's dev build
2. Click on any main menu item
3. Look at main menu pop-up => it's below than should be

Actual Results:
main menu pop-up isn't on its normal place

Expected Results:
main menu pop-up is on its normal place


Reproducible: Always


User Profile Reset: No

Additional Info:
Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: ea3317188589554ea5773db53aa9e0cc88bee271
CPU threads: 16; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win
Locale: ru-RU (ru_RU); UI: en-US
Calc: CL threaded

but not in

Version: 7.5.0.3 (X86_64) / LibreOffice Community
Build ID: c21113d003cd3efa8c53188764377a8272d9d6de
CPU threads: 16; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win
Locale: ru-RU (ru_RU); UI: en-US
Calc: CL threaded

=> regression
Comment 1 Michael Weghorn 2023-07-27 11:13:06 UTC 
I've seen a similar issue on Linux with kf5 and gtk3, but no more with current master.

I didn't check further, but I suspect that it may have been caused by

commit ea3317188589554ea5773db53aa9e0cc88bee271
Author: Noel Grandin
Date:   Tue Jul 25 20:59:19 2023 +0200

    simplify logic in FloatingWindow::ImplCalcPos
    
    no need to convert this position backwards and forwards
    
    Change-Id: Iba4ebcba420025865638c99fa143ea0d7d262805
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154901
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin

and fixed by the corresponding revert:

commit 2b3dcd24e4da1f83bfaf8e0ffd93839eeae621b0
Author: Noel Grandin
Date:   Wed Jul 26 10:35:36 2023 +0200

    Revert "simplify logic in FloatingWindow::ImplCalcPos"
    
    This reverts commit ea3317188589554ea5773db53aa9e0cc88bee271.
    
    Reason for revert: There are 3 different coordinate systems at work here, not 2, my mistake.
    
    Change-Id: Iee69b8ba3c2fe60b26b118e79116035be42c6c45
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154922
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin

Do you still see this with a current daily build/git master?
Comment 2 m_a_riosv 2023-07-27 11:34:01 UTC 
Looks like a dup of tdf#156479, set up now a RESOLVED FIXED.
Comment 3 Roman Kuznetsov 2023-07-27 19:49:34 UTC 
yeah I don't see the problem in

Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: cf8f7b91f41821b79495c0388359c4cb1156ea67
CPU threads: 16; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win
Locale: ru-RU (ru_RU); UI: en-US
Calc: CL threaded

*** This bug has been marked as a duplicate of bug 156479 ***