Bug 483348

Thanks for the bug report. A few questions:

1. Does it also happen on X11, or only on Wayland?
2. Does it also happen if you lock the screen with Meta+L and then unlock, or only on screen sleep/wake?
3. If you create a new clean user account and replicate by hand the same panel setup there, does the bug still happen? Or no?
4. What GPU are these screens plugged into?

Oh and one more too:

5. When this happens, can you make the issue go away by restarting plasmashell with `systemctl restart --user plasma-plasmashell.service` (assuming you're using the systemd boot; if not, try `plasmashell --replace`)?

I'm asking all these questions because I strongly suspect this is Bug 483102, but I need a few more data points to be sure.

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/4108

Created attachment 167458 [details]
Incorrect alignment on X11

Created attachment 167460 [details]
Not working on new account after opening panel edit

Created attachment 167461 [details]
Rounded panel before opening edit

Created attachment 167462 [details]
"Squared" panel after opening edit

Sorry for delay, I was away from the affected system for a week.

(In reply to Nate Graham from comment #1)
> Thanks for the bug report. A few questions:
> 
> 1. Does it also happen on X11, or only on Wayland?
It doesn't show on another screen on X11, but the alignment is incorrect on X11. As visible on https://bugsfiles.kde.org/attachment.cgi?id=167458

> 2. Does it also happen if you lock the screen with Meta+L and then unlock, or only on screen sleep/wake?
That happens only on LOG-IN. Locking, unlocking, screen sleep doesn't change anything.

> 3. If you create a new clean user account and replicate by hand the same panel setup there, does the bug still happen? Or no?
It shows the panel on correct screen, but instantly when I will open panel editing, it will start to stick out to another screen (fill width). This is visible on https://bugsfiles.kde.org/attachment.cgi?id=167460

Also there is a difference in panel shape before and after going to edit. 
Before edit: https://bugs.kde.org/attachment.cgi?id=167461
After edit: https://bugs.kde.org/attachment.cgi?id=167462

I'm not editing anything, I'm just entering edit mode. 

> 4. What GPU are these screens plugged into?
This is AMD 6900 XT with "amdgpu" driver.

> 5. When this happens, can you make the issue go away by restarting plasmashell with `systemctl restart --user plasma-plasmashell.service` (assuming you're using the systemd boot; if not, try `plasmashell --replace`)?

For some reason it has problem with connecting to dbus, so I did `killall plasmashell` `kstart plasmashell`, and the effect is the same like on log-in - the panel is on the wrong screen.

Git commit 27dd68970f33627ba52903c2fc5b1442c7217e0f by David Redondo.
Committed on 26/03/2024 at 13:47.
Pushed by davidre into branch 'master'.

Avoid positioning the panel when it has no size yet

Calling setPosition with a position of the left-most pixel of a screen
and  size of 0x0 will set the screen to be the one to the left of it.
Making screenToFollow() and QWindow::screen() out of sync temporarily.
While arguably a Qt issue and this being fixed up later when the panel
resizes and other properties are updated, we need to workaround this
to our usage of layer-shell. Layer surfaces can't change screen once
created. It can happen that the surface is created during the time
that the panel is on the wrong screen, resulting in the panel
appearing on the wrong screen while thinking it is on another one.
To fix this the initial positionPanel() call is removed, the
panel will now be positioned after it has resized from the event handler.
Related: bug 483102, bug 470434
FIXED-IN:6.0.3

M  +1    -1    shell/panelview.cpp

https://invent.kde.org/plasma/plasma-workspace/-/commit/27dd68970f33627ba52903c2fc5b1442c7217e0f

Git commit d03a98fc69c94f83f9410651d15a21ff41a72904 by David Redondo.
Committed on 26/03/2024 at 14:06.
Pushed by davidre into branch 'Plasma/6.0'.

Avoid positioning the panel when it has no size yet

Calling setPosition with a position of the left-most pixel of a screen
and  size of 0x0 will set the screen to be the one to the left of it.
Making screenToFollow() and QWindow::screen() out of sync temporarily.
While arguably a Qt issue and this being fixed up later when the panel
resizes and other properties are updated, we need to workaround this
to our usage of layer-shell. Layer surfaces can't change screen once
created. It can happen that the surface is created during the time
that the panel is on the wrong screen, resulting in the panel
appearing on the wrong screen while thinking it is on another one.
To fix this the initial positionPanel() call is removed, the
panel will now be positioned after it has resized from the event handler.
Related: bug 483102, bug 470434
FIXED-IN:6.0.3


(cherry picked from commit 27dd68970f33627ba52903c2fc5b1442c7217e0f)

M  +1    -1    shell/panelview.cpp

https://invent.kde.org/plasma/plasma-workspace/-/commit/d03a98fc69c94f83f9410651d15a21ff41a72904

This bug is not fixed in 6.0.3. The problem still persists

I don't have this issue anymore