Bug 144585 - button popups don't open with KF5 VCL on Wayland
Summary: button popups don't open with KF5 VCL on Wayland
Status: REOPENED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
7.3.0.0 alpha0+
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:7.2.2
Keywords: bibisectNotNeeded, regression
: 144993 (view as bug list)
Depends on:
Blocks: Wayland KDE
  Show dependency treegraph
 
Reported: 2021-09-18 10:46 UTC by Antonio Rojas
Modified: 2021-10-12 10:16 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Antonio Rojas 2021-09-18 10:46:40 UTC
Description:
In 7.2.1, and using the KF5 VCL on Wayland, button popus don't open (or rather close immediately, making them unusable).

Like bug 144037, this is a regression from 9dcf5816c90e9819861332f11e014ef7b78e2fe7, but a1a83b6b3b4459de353e0cdc8cdf402370c0a5c4 does NOT fix this.

Steps to Reproduce:
1.Open Writer on Wayland with the KF5 VCL
2. Try to insert a table via the toolbar button

Actual Results:
Button popup disappears

Expected Results:
Button popup opens and can be used


Reproducible: Always


User Profile Reset: Yes


OpenGL enabled: Yes

Additional Info:
Version: 7.2.1.2 / LibreOffice Community
Build ID: 20(Build:2)
CPU threads: 8; OS: Linux 5.14; UI render: default; VCL: kf5 (cairo+wayland)
Locale: es-ES (es_ES.UTF-8); UI: es-ES
7.2.1-2
Calc: threaded
Comment 1 Buovjaga 2021-09-18 17:13:32 UTC
Yeah, they are rather wonky: they do open upon the second click, but are not placed below their parent element.

Arch Linux 64-bit
Version: 7.3.0.0.alpha0+ / LibreOffice Community
Build ID: 88a6ac9f75a4375b72111af5a3e004bef4226455
CPU threads: 8; OS: Linux 5.14; UI render: default; VCL: kf5 (cairo+wayland)
Locale: fi-FI (fi_FI.UTF-8); UI: en-US
Calc: threaded
Built on 15 September 2021
Comment 2 Antonio Rojas 2021-09-19 10:48:54 UTC
In kwin it doesn't open at all, no matter how many times I click.
Comment 3 Buovjaga 2021-09-19 15:42:30 UTC
(In reply to Antonio Rojas from comment #2)
> In kwin it doesn't open at all, no matter how many times I click.

It does for me, just not for you.
Comment 4 Michael Weghorn 2021-09-21 12:27:01 UTC
(In reply to Buovjaga from comment #1)
> Yeah, they are rather wonky: they do open upon the second click, but are not
> placed below their parent element.

That's what I get as well w/ current master in a Plasma Wayland session on Debian testing (plasma-desktop 4:5.21.5-2, kwin-wayland   4:5.21.5-2).
Comment 5 Commit Notification 2021-09-25 12:59:44 UTC
Jan-Marek Glogowski committed a patch related to this issue.
It has been pushed to "libreoffice-7-2":

https://git.libreoffice.org/core/commit/a928bab9fb68ff8fe45aca212b3c794799db31f6

tdf#144585 Qt5 restore old VCL popup handling

It will be available in 7.2.3.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 6 Michael Weghorn 2021-09-25 13:19:45 UTC
Fixed by reverting the involved commits on the 7-2 branch for now. Let's leave this bug report open until there's a fix on the master branch, too.
Comment 7 Commit Notification 2021-09-26 17:48:26 UTC
Jan-Marek Glogowski committed a patch related to this issue.
It has been pushed to "libreoffice-7-2-2":

https://git.libreoffice.org/core/commit/dac5a26ae1108d9ad29e09a800601247fe8c50d6

tdf#144585 Qt5 restore old VCL popup handling

It will be available in 7.2.2.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 8 Michael Weghorn 2021-10-12 10:01:58 UTC
This is fixed in the 7-2 branch, but issue still exists on master (Wayland-only). Reopening the ticket, s.a. discussion in tdf#144037.

@Jan-Marek: Do you think it makes sense to revert the corresponding commits for master as well for now, until a better solution has been found?
Comment 9 Michael Weghorn 2021-10-12 10:16:03 UTC
*** Bug 144993 has been marked as a duplicate of this bug. ***