Fix LayoutCommand

nikitabobko · Oct 29, 2023 · c30fcf9 · c30fcf9
1 parent 526ff35
commit c30fcf9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/docs/commands.md b/docs/commands.md
@@ -89,8 +89,8 @@ layout (h_list|v_list|h_accordion|v_accordion|list|accordion|horizontal|vertical
 
 Changes layout of the focused window to the given layout.
 
-If several arguments are supplied then the first layout that describes the currently active is found. The layout specified after
-the found one will be applied. If the currently active layout is not in the list, the first layout in the list will be activated.
+If several layouts are supplied then finds the first layout that doesn't describe the currently active layout, and applies the
+found layout.
 
 `layout tiling` is the only command that makes the focused floating window tiled.
 

diff --git a/src/command/LayoutCommand.swift b/src/command/LayoutCommand.swift
@@ -18,9 +18,8 @@ struct LayoutCommand: Command {
     func runWithoutLayout() {
         check(Thread.current.isMainThread)
         guard let window = focusedWindowOrEffectivelyFocused else { return }
-        let targetDescription: LayoutDescription = toggleBetween.firstIndex(where: { window.matchesDescription($0) })
-            .flatMap { toggleBetween.getOrNil(atIndex: $0 + 1) }
-            .orElse { toggleBetween.first! }
+        let targetDescription: LayoutDescription = toggleBetween.first(where: { !window.matchesDescription($0) })
+            ?? toggleBetween.first!
         if window.matchesDescription(targetDescription) {
             return
         }