Skip to content

Commit

Permalink
fix(wm): correct border overflow handling logic
Browse files Browse the repository at this point in the history
This commit fixes a regression introduced in the regex rule matching
refactor.

Invisible borders should be removed from applications that are not
identified as border overflow applications, not vice versa.

This is because applications that are overflowing their borders
effectively erase the invisible borders, while applications that are not
overflowing their borders leave the system's invisible borders
visibility intact.

It is this latter group that we should be targeting with the
should_remove_border variable.
  • Loading branch information
LGUG2Z committed Nov 3, 2023
1 parent e9fb929 commit f659deb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion komorebi/src/window.rs
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ impl Window {
let class = &self.class()?;
let exe_name = &self.exe()?;

let should_remove_border = should_act(
let should_remove_border = !should_act(
title,
exe_name,
class,
Expand Down

0 comments on commit f659deb

Please sign in to comment.