From eb2dce1b530584496034bc6d7f14e9b29a487faf Mon Sep 17 00:00:00 2001
From: Ivan Molodetskikh <yalterz@gmail.com>
Date: Fri, 23 Feb 2024 14:40:56 +0400
Subject: Fix default width fixed not being honored with borders

---
 src/layout/mod.rs | 27 ++++++++++++---------------
 1 file changed, 12 insertions(+), 15 deletions(-)

(limited to 'src/layout/mod.rs')

diff --git a/src/layout/mod.rs b/src/layout/mod.rs
index 4fc8021a..61bdaa8d 100644
--- a/src/layout/mod.rs
+++ b/src/layout/mod.rs
@@ -534,13 +534,12 @@ impl<W: LayoutElement> Layout<W> {
         width: Option<ColumnWidth>,
         is_full_width: bool,
     ) -> Option<&Output> {
-        let width = width.unwrap_or_else(|| {
-            let mut width = window.size().w;
+        let mut width = width.unwrap_or_else(|| ColumnWidth::Fixed(window.size().w));
+        if let ColumnWidth::Fixed(w) = &mut width {
             if !self.options.border.off {
-                width += self.options.border.width as i32 * 2;
+                *w += self.options.border.width as i32 * 2;
             }
-            ColumnWidth::Fixed(width)
-        });
+        }
 
         match &mut self.monitor_set {
             MonitorSet::Normal {
@@ -591,13 +590,12 @@ impl<W: LayoutElement> Layout<W> {
         width: Option<ColumnWidth>,
         is_full_width: bool,
     ) -> Option<&Output> {
-        let width = width.unwrap_or_else(|| {
-            let mut width = window.size().w;
+        let mut width = width.unwrap_or_else(|| ColumnWidth::Fixed(window.size().w));
+        if let ColumnWidth::Fixed(w) = &mut width {
             if !self.options.border.off {
-                width += self.options.border.width as i32 * 2;
+                *w += self.options.border.width as i32 * 2;
             }
-            ColumnWidth::Fixed(width)
-        });
+        }
 
         match &mut self.monitor_set {
             MonitorSet::Normal { monitors, .. } => {
@@ -628,13 +626,12 @@ impl<W: LayoutElement> Layout<W> {
         width: Option<ColumnWidth>,
         is_full_width: bool,
     ) {
-        let width = width.unwrap_or_else(|| {
-            let mut width = window.size().w;
+        let mut width = width.unwrap_or_else(|| ColumnWidth::Fixed(window.size().w));
+        if let ColumnWidth::Fixed(w) = &mut width {
             if !self.options.border.off {
-                width += self.options.border.width as i32 * 2;
+                *w += self.options.border.width as i32 * 2;
             }
-            ColumnWidth::Fixed(width)
-        });
+        }
 
         let MonitorSet::Normal {
             monitors,
-- 
cgit