Fix 'border toggle' (it "skipped" 1px border) (Thanks joepd)

fixes #818

diff --git a/src/commands.c b/src/commands.c
index 000dd20835e8f3326f045956bc53036e98a0925b..607e1c11dbaae0b42ba4a75b6eb7d43d61a1e174 100644 (file)
--- a/src/commands.c
+++ b/src/commands.c
@@ -799,11 +799,11 @@ void cmd_border(I3_CMD, char *border_style_str, char *border_width ) {
             border_style++;
             border_style %= 3;
             if (border_style == BS_NORMAL)
-                current->con->current_border_width = 2;
+                tmp_border_width = 2;
             else if (border_style == BS_NONE)
-                current->con->current_border_width = 0;
+                tmp_border_width = 0;
             else if (border_style == BS_PIXEL)
-                current->con->current_border_width = 1;
+                tmp_border_width = 1;
         } else {
             if (strcmp(border_style_str, "normal") == 0)
                 border_style = BS_NORMAL;

diff --git a/testcases/t/169-border-toggle.t b/testcases/t/169-border-toggle.t
index 33f3a8ec58dd1c77b25c286df16f23f52bb54ad8..c89dcc764b9313877e4c087dcd722f36d514904e 100644 (file)
--- a/testcases/t/169-border-toggle.t
+++ b/testcases/t/169-border-toggle.t
@@ -29,25 +29,31 @@ is($nodes[0]->{border}, 'normal', 'border style normal');
 cmd 'border 1pixel';
 @nodes = @{get_ws_content($tmp)};
 is($nodes[0]->{border}, 'pixel', 'border style 1pixel');
+is($nodes[0]->{current_border_width}, 1, 'border width = 1px');
 
 cmd 'border none';
 @nodes = @{get_ws_content($tmp)};
 is($nodes[0]->{border}, 'none', 'border style none');
+is($nodes[0]->{current_border_width}, 0, 'border width = 0px');
 
 cmd 'border normal';
 @nodes = @{get_ws_content($tmp)};
 is($nodes[0]->{border}, 'normal', 'border style back to normal');
+is($nodes[0]->{current_border_width}, 2, 'border width = 2px');
 
 cmd 'border toggle';
 @nodes = @{get_ws_content($tmp)};
 is($nodes[0]->{border}, 'none', 'border style none');
+is($nodes[0]->{current_border_width}, 0, 'border width = 0px');
 
 cmd 'border toggle';
 @nodes = @{get_ws_content($tmp)};
 is($nodes[0]->{border}, 'pixel', 'border style 1pixel');
+is($nodes[0]->{current_border_width}, 1, 'border width = 1px');
 
 cmd 'border toggle';
 @nodes = @{get_ws_content($tmp)};
 is($nodes[0]->{border}, 'normal', 'border style back to normal');
+is($nodes[0]->{current_border_width}, 2, 'border width = 2px');
 
 done_testing;