Skip to content

Commit

Permalink
fix(wm): stop wrongfully removing layout-flip
Browse files Browse the repository at this point in the history
This commit removes the code on the workspace `update` on `layout-rules`
where it was setting the `layout-flip` to `None` if the layout was
different from `BSP`. This appears to be some old code when the
layout-flip would only apply to the `BSP` layout. However now it appears
to apply to all layouts so this code shouldn't exist. This commit also
changes the docs from the `FlipLayout` command to remove the statement
that only applied to `BSP` since it is no longer true.
  • Loading branch information
alex-ds13 authored and LGUG2Z committed Jan 23, 2025
1 parent e153d2e commit 39621c1
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
4 changes: 0 additions & 4 deletions komorebi/src/workspace.rs
Original file line number Diff line number Diff line change
Expand Up @@ -333,10 +333,6 @@ impl Workspace {
}

if let Some(updated_layout) = updated_layout {
if !matches!(updated_layout, Layout::Default(DefaultLayout::BSP)) {
self.set_layout_flip(None);
}

self.set_layout(updated_layout);
}
}
Expand Down
2 changes: 1 addition & 1 deletion komorebic/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1175,7 +1175,7 @@ enum SubCommand {
#[clap(hide = true)]
#[clap(arg_required_else_help = true)]
LoadCustomLayout(LoadCustomLayout),
/// Flip the layout on the focused workspace (BSP only)
/// Flip the layout on the focused workspace
#[clap(arg_required_else_help = true)]
FlipLayout(FlipLayout),
/// Promote the focused window to the top of the tree
Expand Down

0 comments on commit 39621c1

Please sign in to comment.