From: Johan Malm <jgm323@gmail.com>
Date: Sun, 19 May 2024 15:50:53 +0000 (+0100)
Subject: Deprecate MoveToCursor
X-Git-Url: https://git.mdlowis.com/?a=commitdiff_plain;h=bab1be834d3bc526e1ae75ea2d68cf9969b92108;p=proto%2Flabwc.git

Deprecate MoveToCursor

Use <action name="AutoPlace" policy="cursor"/> instead.

Related-to: #1785
Suggested-by: @tokyo4j
---

diff --git a/docs/labwc-actions.5.scd b/docs/labwc-actions.5.scd
index 097cd5c2..703236ef 100644
--- a/docs/labwc-actions.5.scd
+++ b/docs/labwc-actions.5.scd
@@ -83,6 +83,9 @@ Actions are used in menus and keyboard/mouse bindings.
 *<action name="MoveToCursor" />*
 	Move to be centered on cursor.
 	Tries to prevent any part of the window from going off-screen.
+	This action is deprecated from v0.7.3. To ensure your config works in
+	future labwc releases, please use:
+	*<action name="AutoPlace" policy="cursor">*
 
 *<action name="MoveRelative" x="" y="" />*
 	Move window relative to its current position. Positive value of x moves
diff --git a/src/action.c b/src/action.c
index db2b7557..3ed88d68 100644
--- a/src/action.c
+++ b/src/action.c
@@ -904,6 +904,9 @@ actions_run(struct view *activator, struct server *server,
 			}
 			break;
 		case ACTION_TYPE_MOVETO_CURSOR:
+			wlr_log(WLR_ERROR,
+				"Action MoveToCursor is deprecated. To ensure your config works in future labwc "
+				"releases, please use <action name=\"AutoPlace\" policy=\"cursor\">");
 			if (view) {
 				view_move_to_cursor(view);
 			}