Bug 156483 - main menu pop-up isn't on its normal place
Summary: main menu pop-up isn't on its normal place
Status: RESOLVED DUPLICATE of bug 156479
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
24.2.0.0 alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisectRequest, regression
Depends on:
Blocks: Main-Menu
  Show dependency tree / graph
 
Reported: 2023-07-26 20:05 UTC by Roman Kuznetsov
Modified: 2023-07-27 19:49 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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 ***