Bug 459523

Summary: Plasmoid popups now displace windows instead of appearing on top
Product: [Frameworks and Libraries] kwayland Reporter: Iyán Méndez Veiga <me>
Component: generalAssignee: Martin Flöser <mgraesslin>
Status: RESOLVED FIXED    
Severity: normal CC: arojas, bizyaev, bugseforuns, kde, kwin-bugs-null, nate
Priority: VHI Keywords: regression
Version: 5.98.0   
Target Milestone: ---   
Platform: Other   
OS: Other   
See Also: https://bugs.kde.org/show_bug.cgi?id=459188
https://bugs.kde.org/show_bug.cgi?id=459558
Latest Commit: Version Fixed In: 5.98 re-spin
Sentry Crash Report:

Description Iyán Méndez Veiga 2022-09-22 13:22:16 UTC
SUMMARY
When opening the calendar applet, it displaces all windows on its way instead of appearing on top of them.


STEPS TO REPRODUCE
1. Maximize a window
2. Open the calendar applet

OBSERVED RESULT
Window is resized by the calendar

EXPECTED RESULT
Calendar should appear on top and windows should not be resized

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.25.90
KDE Frameworks Version: 5.98.0
Qt Version: 5.15.6
Kernel Version: 5.19.10-arch1-1 (64-bit)
Graphics Platform: Wayland
Processors: 8 × 11th Gen Intel® Core™ i5-1135G7 @ 2.40GHz
Memory: 15.4 GiB of RAM
Graphics Processor: Mesa Intel® Xe Graphics
Manufacturer: LENOVO
Product Name: 20XYCTO1WW
System Version: ThinkPad X1 Yoga Gen 6
Comment 1 Vlad Zahorodnii 2022-09-22 13:33:52 UTC
Can you reproduce it when running plasma 5.25.0?
Comment 2 Iyán Méndez Veiga 2022-09-22 13:39:21 UTC
I don't think this was happening on 5.25 or I would have noticed, but I'm not 100% sure. Also, I'm not sure it happens on X11 as well. I will check later today on my desktop.
Comment 3 Nate Graham 2022-09-22 20:41:57 UTC
Almost positive this will end up being caused by the same thing that caused Bug 459188, and will be fixed by its fix.

*** This bug has been marked as a duplicate of bug 459188 ***
Comment 4 Antonio Rojas 2022-09-22 20:54:21 UTC
(In reply to Nate Graham from comment #3)
> Almost positive this will end up being caused by the same thing that caused
> Bug 459188, and will be fixed by its fix.
> 
> *** This bug has been marked as a duplicate of bug 459188 ***

Actually, it's the other way around. This is *caused* by the fix for bug 459188
Comment 5 Nate Graham 2022-09-22 20:56:51 UTC
oh god
Comment 6 Bug Janitor Service 2022-09-22 21:21:04 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kwayland/-/merge_requests/79
Comment 7 Vlad Zahorodnii 2022-09-23 08:53:36 UTC
Can you clarify what compositor do you use?
Comment 8 Zamundaaa 2022-09-23 09:01:54 UTC
Git commit a59a303291829747fcb5cdd2d8dac5514772672c by Xaver Hugl.
Committed on 22/09/2022 at 21:20.
Pushed by vladz into branch 'master'.

client/plasmashell: fix applet popups displacing other windows

M  +1    -0    src/client/plasmashell.cpp

https://invent.kde.org/frameworks/kwayland/commit/a59a303291829747fcb5cdd2d8dac5514772672c
Comment 9 Vlad Zahorodnii 2022-09-23 09:03:09 UTC
(In reply to Vlad Zahorodnii from comment #7)
> Can you clarify what compositor do you use?

Nvm, I was confused by 5.25.90. 5.25.90 would still fallback to the panel type. Only final 5.26.0 should use the appletpopup type.
Comment 10 Iyán Méndez Veiga 2022-09-23 09:30:10 UTC
One thing that I noticed now is that the issue does not affect xwayland apps. For those, the calendar is still on top of the app.
Comment 11 Nate Graham 2022-09-23 14:06:32 UTC
Can anyone who was affected by this verify that that last commit fixed things for them and doesn't regress anything else? Thanks!
Comment 12 Nate Graham 2022-09-23 15:23:55 UTC
*** Bug 459565 has been marked as a duplicate of this bug. ***
Comment 13 Iyán Méndez Veiga 2022-09-25 12:29:50 UTC
I confirm that the issue is solved for me after applying the original patch (https://invent.kde.org/frameworks/kwayland/-/commit/d02188ad.patch) and the fix to that path (https://invent.kde.org/frameworks/kwayland/-/commit/a59a3032.patch) on top of tag v5.98.0.
Comment 14 Patrick Silva 2022-09-25 12:33:00 UTC
*** Bug 459614 has been marked as a duplicate of this bug. ***
Comment 15 Iyán Méndez Veiga 2022-09-25 12:39:50 UTC
Unfortunately, I do observe another regression with both patches. Notifications are drawn partially on top of the panel instead of leaving some margin. Removing both patches solves the issue for me.
Comment 16 Nate Graham 2022-09-26 16:41:03 UTC
Sigh, let's get another bug report to track that new issue. What a game of whack-a-mole...

Thanks for your patience here!