Bug 437163

Summary: Menu of yakuake opens in wrong position and has window decoration on Wayland, when tray icon is enabled
Product: [Applications] yakuake Reporter: Patrick Silva <bugseforuns>
Component: generalAssignee: Eike Hein <hein>
Status: RESOLVED WORKSFORME    
Severity: normal CC: aspotashev, nicolas.fella, oded
Priority: NOR    
Version: 23.04.0   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: screencast showing repro failing
reproducing the bug on neon unstable
screenshot of version 24.01.75

Description Patrick Silva 2021-05-15 16:48:40 UTC
STEPS TO REPRODUCE
1. minimize all windows
2. open yakuake
3. click on desktop
4. click on second button in the lower right corner

OBSERVED RESULT
menu opens in wrong position and has window decoration

EXPECTED RESULT
menu opens in correct position without window decoration

SOFTWARE/OS VERSIONS
yakuake 21.04.1
Operating System: Arch Linux
KDE Plasma Version: 5.21.90
KDE Frameworks Version: 5.82.0
Qt Version: 5.15.2
Graphics Platform: Wayland
Comment 1 Oded Arbel 2022-12-01 10:09:33 UTC
Cannot reproduce with Yakuake 23.03.70 on Plasma 5.26.
Comment 2 Patrick Silva 2022-12-01 10:14:54 UTC
I can reproduce on neon unstable.

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.26.80
KDE Frameworks Version: 5.101.0
Qt Version: 5.15.7
Graphics Platform: Wayland
Processors: 4 × Intel® Core™ i3-3110M CPU @ 2.40GHz
Comment 3 Oded Arbel 2022-12-01 11:33:49 UTC
Created attachment 154189 [details]
screencast showing repro failing

Then I may not understand what the issue is.

Running on the same Neon unstable, here's a video of me trying to perform the repro and getting the expected result (i.e. the repro failed).

Note: as can be seen in the video, that you can only click on the desktop and have the Yakuake window stay open is by pinning it ("keep window open when it loses focus") which is currently somewhat problematic under Wayland due to other bugs already reported.
Comment 4 Patrick Silva 2022-12-01 11:51:02 UTC
Created attachment 154190 [details]
reproducing the bug on neon unstable

Here is my video showing the bug.
Comment 5 Oded Arbel 2022-12-01 13:15:51 UTC
😲
So I noticed that you use the tray icon to open the window - which I don't have enabled. So I enabled the tray icon and ran the repro again - and now I cannot not reproduce it: with other windows on the desktop; triggered from shortcut; triggered from mouse gestures, unpinned - it keeps showing the menu in a window. Then, turn off the system tray icon - and now it can't be reproduced anymore.

This is definitely about the system tray icon. I'll take a closer look at it this evening or over the weekend - when I have time.
Comment 6 Patrick Silva 2022-12-01 13:25:36 UTC
I can also reproduce on both neon unstable and Arch Linux (yakuake 22.12 RC) following these steps:
1. open yakuake via krunner
2. make sure yakuake is pinned
3. click on desktop
4. click on the second button in the lower right corner
Comment 7 Patrick Silva 2022-12-01 13:33:14 UTC
And I'm unable to reproduce on both systems if the system tray icon is disabled in yakuake settings.
Comment 8 Bug Janitor Service 2022-12-01 19:59:49 UTC
A possibly relevant merge request was started @ https://invent.kde.org/utilities/yakuake/-/merge_requests/86
Comment 9 Patrick Silva 2023-11-13 19:45:08 UTC
Created attachment 163130 [details]
screenshot of version 24.01.75

Can reproduce with version 24.01.75.

1. make sure the system tray is enabled in yakuake settings
2. minimize all windows
3. open yakuake
4. pin yakuake
5. click on desktop
6. right-click on yakuake

Reslt: context menu with window decoration opens in the middle of the screen. Please see the attached screenshot.
Comment 10 Oded Arbel 2023-11-13 23:58:22 UTC
(In reply to Patrick Silva from comment #9)
> Can reproduce with version 24.01.75.

The merge request linked in comment #8 (MR!86) is supposed to fix this, but is still open and hadn't been merged. If you want to see this fixed - maybe add a comment on the MR or something.
Comment 11 Nicolas Fella 2023-11-14 00:18:52 UTC
I can reproduce this for the context/right click menu, but not the "main" menu with current master
Comment 12 Nicolas Fella 2023-11-14 00:23:23 UTC
I can reproduce it with 23.08.1 for the main menu.

It's plausible that using Qt6 fixed it
Comment 13 Oded Arbel 2023-12-10 21:15:52 UTC
I can no longer reproduce the issue on Plasma 5.27.10, with the current Yakuake package from Neon testing.

I'm closing the MR as it appears to be no longer needed.
Comment 14 Patrick Silva 2024-02-16 20:50:49 UTC
Can reproduce with Yakuake 24.01.95 by following the steps from comment 9.
Comment 15 Patrick Silva 2024-02-16 20:51:19 UTC
Operating System: Arch Linux 
KDE Plasma Version: 5.93.0
KDE Frameworks Version: 5.249.0
Qt Version: 6.7.0 beta3
Kernel Version: 6.7.4-arch1-1 (64-bit)
Graphics Platform: Wayland
Comment 16 Patrick Silva 2024-08-01 00:07:17 UTC
Cannot reproduce with yakuake 24.05.2.

Operating System: Arch Linux 
KDE Plasma Version: 6.1.3
KDE Frameworks Version: 6.4.0
Qt Version: 6.7.2
Graphics Platform: Wayland