Skip to content

[Switch presets] add (no-)wraparound & make cycle-back an OPTION#3571

Open
two-horned wants to merge 3 commits intoniri-wm:mainfrom
two-horned:no-wraparound
Open

[Switch presets] add (no-)wraparound & make cycle-back an OPTION#3571
two-horned wants to merge 3 commits intoniri-wm:mainfrom
two-horned:no-wraparound

Conversation

@two-horned
Copy link
Contributor

Improves #1670 and adds a no-wraparound option which desire was mentioned at couple of places,
like in #1000 and #1367.

I didn't see the need to add tests, so if someone wants to contribute in that regard, please do so.

@two-horned two-horned changed the title Add wraparound + make cycle-back an OPTION [Switch presets] add (no-)wraparound & make cycle-back an OPTION Mar 7, 2026
@two-horned
Copy link
Contributor Author

two-horned commented Mar 7, 2026

Important note: This will break configs using old switch-preset-[column/window]-width-back or switch-preset-window-height-back commands. You are now supposed to use optional arguments instead.

Quick examples:

bind {
  Mod+R { switch-preset-column-width-back; } // defaults are forwards=true, wrap=true
  Mod+R { switch-preset-column-width-back wrap=true; }
  Mod+R { switch-preset-column-width-back forwards=false; }
  Mod+R { switch-preset-column-width-back forwards=false wrap=false; }
}

@two-horned two-horned marked this pull request as ready for review March 7, 2026 15:03
@HigherOrderLogic
Copy link
Contributor

This will break configs using old

Sounds like a big no no for this to get merged.

@Sempyos Sempyos added area:layout Columns, workspaces, scrolling, fullscreen, resize area:config Config parsing, default config, new settings pr kind:feature New features and functionality labels Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:config Config parsing, default config, new settings area:layout Columns, workspaces, scrolling, fullscreen, resize pr kind:feature New features and functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants