From ecb10944d3776e4916d3317cb36e611fa1e79e0f Mon Sep 17 00:00:00 2001
From: tokyo4j <hrak1529@gmail.com>
Date: Thu, 14 Nov 2024 06:12:27 +0900
Subject: [PATCH] placement: set default placement policy to cascade

---
 docs/labwc-config.5.scd | 2 +-
 docs/rc.xml.all         | 2 +-
 src/config/rcxml.c      | 8 ++++----
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/docs/labwc-config.5.scd b/docs/labwc-config.5.scd
index 66a8d611..9d469dde 100644
--- a/docs/labwc-config.5.scd
+++ b/docs/labwc-config.5.scd
@@ -232,7 +232,7 @@ this is for compatibility with Openbox.
 	center new windows under the cursor. The "cascade" policy will try to
 	place new windows at the center of the active output, but possibly
 	shifts its position to bottom-right not to cover existing windows.
-	Default is "center".
+	Default is "cascade".
 
 *<placement><cascadeOffset><x>*++
 *<placement><cascadeOffset><y>*
diff --git a/docs/rc.xml.all b/docs/rc.xml.all
index b1a8bae6..addb03d4 100644
--- a/docs/rc.xml.all
+++ b/docs/rc.xml.all
@@ -17,7 +17,7 @@
   </core>
 
   <placement>
-    <policy>center</policy>
+    <policy>cascade</policy>
     <!--
       When <placement><policy> is "cascade", the offset for cascading new
       windows can be overwritten like this:
diff --git a/src/config/rcxml.c b/src/config/rcxml.c
index e4f6426e..3951c72f 100644
--- a/src/config/rcxml.c
+++ b/src/config/rcxml.c
@@ -1079,9 +1079,9 @@ entry(xmlNode *node, char *nodename, char *content)
 	} else if (!strcasecmp(nodename, "reuseOutputMode.core")) {
 		set_bool(content, &rc.reuse_output_mode);
 	} else if (!strcmp(nodename, "policy.placement")) {
-		rc.placement_policy = view_placement_parse(content);
-		if (rc.placement_policy == LAB_PLACE_INVALID) {
-			rc.placement_policy = LAB_PLACE_CENTER;
+		enum view_placement_policy policy = view_placement_parse(content);
+		if (policy != LAB_PLACE_INVALID) {
+			rc.placement_policy = policy;
 		}
 	} else if (!strcasecmp(nodename, "xwaylandPersistence.core")) {
 		set_bool(content, &rc.xwayland_persistence);
@@ -1434,7 +1434,7 @@ rcxml_init(void)
 	}
 	has_run = true;
 
-	rc.placement_policy = LAB_PLACE_CENTER;
+	rc.placement_policy = LAB_PLACE_CASCADE;
 	rc.placement_cascade_offset_x = 0;
 	rc.placement_cascade_offset_y = 0;