inconsistent behaviour for `cyclenext prev` between normal and `fullscreen 1` mode

[m4rch3n1ng]

Hyprland Version

Hyprland, built from branch main at commit a9b8e2159c2813a6951d0b0186fbb7f2f8554d3e dirty (renderer: reset scissor after renderTexturePrimitive). Tag: v0.27.2-54-ga9b8e215

Bug or Regression?

Bug

Description

when dispatching cyclenext prev with fullscreen 1, it cycles between the last two used windows, but without fullscreen 1 it cycles through all the windows in reverse order to cyclenext. just cyclenext is consistent and works as expected

ideally i think this should be split up into two features: cyclenext prev to cycle between the last two used windows and cyclenext back (or something) to cycle through all windows in reverse order.

How to reproduce

dispatch cyclenext prev (multiple times) with more than two windows open. toggle fullscreen 1 and do the same

Crash reports, logs, images, videos

https://github.com/hyprwm/Hyprland/assets/63159454/efcae516-e9db-49b2-a94e-a83c070805b1

(what's happening in the video: i dispatch cyclenext 8 times, then dispatch cyclenext prev 8 times. i switch into fullscreen 1 mode and do the same. between every switch i wait a second).

relevant hyprland.conf:

$mainMod = SUPER

bind = $mainMod, F, fullscreen, 1
bind = $mainMod, Tab, cyclenext,
bind = $mainMod SHIFT, Tab, cyclenext, prev

[alex-courtis]

Confirmed. Reproduction example:

1. 3 windows on stack
2. fullscreen from middle window on stack
3. cyclenext goes up the stack instead of down
4. windows are now out of order

https://github.com/hyprwm/Hyprland/assets/1505378/a7c3180f-c1ad-4d7d-a340-a5012e2bc5c9

[izmyname]

Happens for me on the latest git.

[Smithx10]

Happening to me also.

[tbeitter]

I am pretty new to hyprland and this was one of the first issues I stumbled upon. Thanks for raising the issue, a fix would be superb :+1: