From: tokyo4j <hrak1529@gmail.com>
Date: Wed, 13 Nov 2024 21:12:27 +0000 (+0900)
Subject: placement: set default placement policy to cascade
X-Git-Url: https://git.mdlowis.com/?a=commitdiff_plain;h=22eefa32a2914ac80a042f748553b5bcb1a962d9;p=proto%2Flabwc.git

placement: set default placement policy to cascade
---

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;