smart_borders: separate smartness from edge types

5 files changed, 17 insertions, 16 deletions

diff --git a/sway/commands/hide_edge_borders.c b/sway/commands/hide_edge_borders.c
index f69bece1..9a1d8445 100644
--- a/sway/commands/hide_edge_borders.c
+++ b/sway/commands/hide_edge_borders.c
@@ -32,14 +32,15 @@ struct cmd_results *cmd_hide_edge_borders(int argc, char **argv) {
 	} else if (strcmp(argv[0], "both") == 0) {
 		config->hide_edge_borders = E_BOTH;
 	} else if (strcmp(argv[0], "smart") == 0) {
-		config->hide_edge_borders = E_SMART;
+		config->hide_edge_borders = E_NONE;
+		config->hide_edge_borders_smart = ESMART_ON;
 	} else if (strcmp(argv[0], "smart_no_gaps") == 0) {
-		config->hide_edge_borders = E_SMART_NO_GAPS;
+		config->hide_edge_borders = E_NONE;
+		config->hide_edge_borders_smart = ESMART_NO_GAPS;
 	} else {
 		return cmd_results_new(CMD_INVALID, expected_syntax);
 	}
 	config->hide_lone_tab = hide_lone_tab;
-	config->saved_edge_borders = config->hide_edge_borders;
 
 	arrange_root();
 
diff --git a/sway/commands/smart_borders.c b/sway/commands/smart_borders.c
index be346106..73878679 100644
--- a/sway/commands/smart_borders.c
+++ b/sway/commands/smart_borders.c
@@ -10,14 +10,12 @@ struct cmd_results *cmd_smart_borders(int argc, char **argv) {
 		return error;
 	}
 
-	enum edge_border_types saved = config->hide_edge_borders;
 	if (strcmp(argv[0], "no_gaps") == 0) {
-		config->hide_edge_borders = E_SMART_NO_GAPS;
+		config->hide_edge_borders_smart = ESMART_NO_GAPS;
 	} else {
-		config->hide_edge_borders = parse_boolean(argv[0], true) ?
-			E_SMART : config->saved_edge_borders;
+		config->hide_edge_borders_smart = parse_boolean(argv[0], true) ?
+			ESMART_ON : ESMART_OFF;
 	}
-	config->saved_edge_borders = saved;
 
 	arrange_root();
 
diff --git a/sway/config.c b/sway/config.c
index afc60a42..1f03610a 100644
--- a/sway/config.c
+++ b/sway/config.c
@@ -296,7 +296,7 @@ static void config_defaults(struct sway_config *config) {
 	config->border_thickness = 2;
 	config->floating_border_thickness = 2;
 	config->hide_edge_borders = E_NONE;
-	config->saved_edge_borders = E_NONE;
+	config->hide_edge_borders_smart = ESMART_OFF;
 	config->hide_lone_tab = false;
 
 	// border colors
diff --git a/sway/sway.5.scd b/sway/sway.5.scd
index 8315f8a1..9119b379 100644
--- a/sway/sway.5.scd
+++ b/sway/sway.5.scd
@@ -603,8 +603,10 @@ The default colors are:
 
 *hide_edge_borders* [--i3] none|vertical|horizontal|both|smart|smart_no_gaps
 	Hides window borders adjacent to the screen edges. Default is _none_. The
-	_--i3_ option enables i3-compatible behavior to hide the title bar on tabbed
-	and stacked containers with one child.
+	_--i3_ option enables i3-compatible behavior to hide the title bar on
+	tabbed and stacked containers with one child. The _smart_|_smart_no_gaps_
+	options are equivalent to setting _smart_borders_ smart|no_gaps and
+	_hide_edge_borders_ none.
 
 *input* <input_device> <input-subcommands...>
 	For details on input subcommands, see *sway-input*(5).
@@ -621,9 +623,9 @@ The default colors are:
 
 *smart_borders* on|no_gaps|off
 	If smart_borders are _on_, borders will only be enabled if the workspace
-	has more than one visible child (identical to _hide_edge_borders_ smart).
-	If smart_borders is set to _no_gaps_, borders will only be enabled if the
-	workspace has more than one visible child and gaps equal to zero.
+	has more than one visible child. If smart_borders is set to _no_gaps_,
+	borders will only be enabled if the workspace has more than one visible
+	child and gaps equal to zero.
 
 *smart_gaps* on|off
 	If smart_gaps are _on_ gaps will only be enabled if a workspace has more
diff --git a/sway/tree/view.c b/sway/tree/view.c
index 376cc332..93d4fefc 100644
--- a/sway/tree/view.c
+++ b/sway/tree/view.c
@@ -215,8 +215,8 @@ void view_autoconfigure(struct sway_view *view) {
 
 	if (!container_is_floating(con) && ws) {
 
-		bool smart = config->hide_edge_borders == E_SMART ||
-			(config->hide_edge_borders == E_SMART_NO_GAPS &&
+		bool smart = config->hide_edge_borders_smart == ESMART_ON ||
+			(config->hide_edge_borders_smart == ESMART_NO_GAPS &&
 			!gaps_to_edge(view));
 		bool hide_smart = smart && view_is_only_visible(view);