Bug 457252 - All plasmashell sub-windows(kickstart, menus, notifications) appear in the upper left corner rather than where they should be
Summary: All plasmashell sub-windows(kickstart, menus, notifications) appear in the up...
Status: RESOLVED DOWNSTREAM
Alias: None
Product: plasmashell
Classification: Plasma
Component: general (show other bugs)
Version: 5.25.3
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-07-29 02:28 UTC by 322997am
Modified: 2022-07-29 12:24 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:
hamilton.tim: Wayland+


Attachments
screenshot of the effect (238.15 KB, image/png)
2022-07-29 02:28 UTC, 322997am
Details
plasmashell log with WAYLAND_DEBUG=1 (324.25 KB, text/plain)
2022-07-29 11:32 UTC, 322997am
Details

Note You need to log in before you can comment on or make changes to this bug.
Description 322997am 2022-07-29 02:28:38 UTC
Created attachment 150963 [details]
screenshot of the effect

SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1. Update to plasma 5.25.3
2. Open any plasmashell window such as by pressing the windows key
3. 

OBSERVED RESULT
Window opens always in the upper left corner

EXPECTED RESULT
Window opens where expected

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma:
(available in About System)
KDE Plasma Version:  5.25.3
KDE Frameworks Version: 5.96.0
Qt Version: 5.15.5

ADDITIONAL INFORMATION
Comment 1 Tim 2022-07-29 10:48:20 UTC
Confirm that this bug also affects me when using a Wayland session. It does not affect X11.

SOFTWARE/OS VERSIONS
Linux: Garuda (Arch)
KDE Plasma Version:  5.25.3
KDE Frameworks Version: 5.96.0
Qt Version: 5.15.5
Graphics Platform: Wayland
Comment 2 David Edmundson 2022-07-29 10:51:39 UTC
please attach output of "WAYLAND_DEBUG=1 plasmashell --replace"

and recreating this situation.
Comment 3 David Edmundson 2022-07-29 10:53:11 UTC
Edit. Don't.

Please shout aggressively at Arch. 

https://github.com/archlinux/svntogit-packages/blob/packages/qt5-wayland/trunk/PKGBUILD#L29
  git cherry-pick -n de46b3c6d1bef4b8f64c7db7069c8fd33ef23ed9 # Keep toplevel windows in the top left corner of the screen
Comment 4 Tim 2022-07-29 11:01:53 UTC
Shouting at Arch commenced.
Comment 5 Antonio Rojas 2022-07-29 11:17:23 UTC
Please explain why a Fedora user should "shout aggressively at Arch"?
Comment 6 322997am 2022-07-29 11:32:42 UTC
Created attachment 150975 [details]
plasmashell log with WAYLAND_DEBUG=1
Comment 7 322997am 2022-07-29 11:33:22 UTC
I forgot to mention, I use a copr repository(zawertun/kde) that gets me the latest KDE release with 0 modifications. So this is definitely something with KDE, not Fedora package maintainers.
Comment 8 322997am 2022-07-29 11:38:41 UTC
I'm gonna change the status, I don't think this copr has that commit from the arch maintainers.
Comment 9 David Edmundson 2022-07-29 11:59:01 UTC
@Antonio, I should have phrased my comment much better I apologise for that.

WRT arch packaging, we're backporting something which will cause known breakages. Even if we also patch out plasmashell, there's still kscreen OSDs that won't get fixed till 5.26.0. I left a comment on the github link above.

As to why it affects Fedora, can you link me to Fedora's COPR  Qt packaging?
Comment 10 David Edmundson 2022-07-29 12:00:53 UTC
The log shows plasma is explicitly requesting this position.

[4282679.397]  -> org_kde_plasma_surface@119.set_position(0, 0)
Comment 11 322997am 2022-07-29 12:04:41 UTC
(In reply to David Edmundson from comment #9)
> @Antonio, I should have phrased my comment much better I apologise for that.
> 
> WRT arch packaging, we're backporting something which will cause known
> breakages. Even if we also patch out plasmashell, there's still kscreen OSDs
> that won't get fixed till 5.26.0. I left a comment on the github link above.
> 
> As to why it affects Fedora, can you link me to Fedora's COPR  Qt packaging?

Here is a link to the newest KDE repo: https://copr.fedorainfracloud.org/coprs/zawertun/kde/packages/. It does not seem to package Qt at all, that means that Qt5 is gotten from here: https://src.fedoraproject.org/rpms/qt5.
Comment 12 Jan Grulich 2022-07-29 12:11:49 UTC
This issue happens because Fedora qt5-qtwayland package has following patch:
https://src.fedoraproject.org/rpms/qt5-qtwayland/blob/rawhide/f/0100-Keep-toplevel-windows-in-the-top-left-corner-of-the-screen.patch

But in order to not break Plasma sessions, we also have a patch in plasma-workspace:
https://src.fedoraproject.org/rpms/plasma-workspace/blob/rawhide/f/disable-toplevel-fixed-positions-on-wayland.patch

Apparently the plasma-workspace package in the mentioned COPR repository doesn't have this fix and therefore using the official qt5-qtwayland package breaks it.
Comment 13 David Edmundson 2022-07-29 12:17:40 UTC
It seems arch also backports the relevant plasma-workspace commit: https://github.com/archlinux/svntogit-packages/tree/packages/plasma-workspace/trunk

Presumably it's a similar situation with " Garuda (Arch)" a Qt from Arch with a backport and a plasma-workspace that isn't.

Ultimately it is good to see the patch backported and it does seem both official Fedora and Arch have gone out their way to backport the plasma-workspace patch which addresses the bulk of any fallout.  When we got this bug report I jumped to a mistaken conclusion on that. That's my bad. 

This issue is still cherry-picking related rather than KDE so I'll close it, even though it seems it's not just a main distro thing.
Comment 14 David Edmundson 2022-07-29 12:20:07 UTC
FWIW I backported the p-w commit into what will be 5.25.4 . Hopefully that will make it safer
Comment 15 322997am 2022-07-29 12:24:12 UTC
I just disabled the copr because fedora seems to have started shipping up to date KDE versions. All is good now for me at least.