Scrolling layout fullscreen issues

[ern775]

On the latest git, I imagine since #14190 was merged, there are some minor issues with the scrolling layout.

(I show mpv in the videos but this happens with all windows.)

First, when exiting fullscreen, the window doesn't seem to go to the correct place:

untitled.mp4

The old behaviour would've been that both windows are visible on screen no matter which one exited fullscreen.

Second, when switching workspaces to a fullscreen window, the behaviour isn't consistent with when fullscreening the focused window:

untitled_.2.mp4

There are some things to note:

• The biggest issue is the ghost window that appears for a second when switching to another workspace from the fullscreen window. This doesn't happen if you fullscreen the focused window then switch, it only happens when switching to a workspace with a fullscreen window then switching back.
• The quickshell bar not hiding when switching workspaces to a fullscreen window.

I believe these are both due to the floating tag. If I un-float the fullscreen window manually, both issues disappear, if I make it floating, they come back.
Idk if this is intended but it is a regression imo, using video players regularly causes these bugs, and these weren't an issue before.

Also I just wanna add that the fullscreen scrolls are very cool and I appreciate these changes overall.

[jam-jarr]

Also experiencing these issues in 0.55 release. I also really like the fullscreen scrolls, but the misbehavior when switching workspaces is very interruptive.

[ern775]

I want to add that on a45fc64 (not fixed on this specific commit, it just happens to be the version I'm on), the second part of this is fixed. However the first part about windows not going to the correct place remains.

[tekstryder]

Be sure to follow and test this PR:

• scrolling/fullscreen: Fix fullscreen/maximise behaviour, fix various existing bugs with scrolling fullscreen #14387

[theRealCarneiro]

Unfortunately, this PR doesn’t seem to resolve the issues that were reported. At least not the first one.

[jam-jarr]

Unfortunately, this PR doesn’t seem to resolve the issues that were reported. At least not the first one.

@theRealCarneiro
You might wanna go comment on the PR then 😭. It's already merged to main

[theRealCarneiro]

@jam-jarr It looks like this PR was intended to fix other issues, so there may not be anything to address there.