[BUG]: CycleLayout doesn't wrap around correctly #680

New Issue

2026-01-05T14:52:22+01:00

adam commented

2026-01-05 14:52:22 +01:00

Originally created by @mosamadeeb on GitHub (Sep 21, 2025).

Summary

When the current layout is Scrolling, cycling next layout changes it to BSP. Cycling to the previous layout from BSP however changes to RightMainVerticalStack instead of Scrolling.

Changing this line to map to Scrolling instead of RightMainVerticalStack will fix the issue.

Version Information

komorebic 0.1.38
tag:v0.1.38
commit_hash:ffa76ea2
build_time:2025-09-13 18:40:25 +00:00
build_env:rustc 1.89.0 (29483883e 2025-08-04),stable-x86_64-pc-windows-msvc

Komorebi Configuration

Hotkey Configuration

Output of komorebic check

Originally created by @mosamadeeb on GitHub (Sep 21, 2025). ### Summary When the current layout is Scrolling, cycling next layout changes it to BSP. Cycling to the previous layout from BSP however changes to RightMainVerticalStack instead of Scrolling. Changing [this line](https://github.com/LGUG2Z/komorebi/blob/160cb7202d41197f3509db9230860f20a29ecd61/komorebi/src/core/default_layout.rs#L206) to map to `Scrolling` instead of `RightMainVerticalStack` will fix the issue. ### Version Information ``` komorebic 0.1.38 tag:v0.1.38 commit_hash:ffa76ea2 build_time:2025-09-13 18:40:25 +00:00 build_env:rustc 1.89.0 (29483883e 2025-08-04),stable-x86_64-pc-windows-msvc ``` ### Komorebi Configuration ```json - ``` ### Hotkey Configuration - ### Output of komorebic check -

adam added the bug label 2026-01-05 14:52:22 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/komorebi#680