From: Johan Malm <jgm323@gmail.com>
Date: Mon, 23 Dec 2024 21:29:46 +0000 (+0000)
Subject: theme: demote padding.height warning
X-Git-Url: https://git.mdlowis.com/?a=commitdiff_plain;h=af978c7ad75309b6a383907b8fd29c09064766ef;p=proto%2Flabwc.git

theme: demote padding.height warning

...from WLR_ERROR to WLR_INFO to avoid 'false' logging when Openbox themes
contain the padding.height settings. Whilst this option is not supported,
the visual appearance is very close to that of Openbox with common themes.

The background to this is that Labwc theme specification has diverged
slightly from that of Openbox with respect to titlebar padding to support
more contemporary looks whilst avoiding breaking changes. For full details
see commit: e16e78e7a4a6e40c8184edd7bb483b0f395482d0

Suggested-by: @Consolatis
---

diff --git a/src/theme.c b/src/theme.c
index a0853d7c..8b88baf3 100644
--- a/src/theme.c
+++ b/src/theme.c
@@ -637,7 +637,7 @@ entry(struct theme *theme, const char *key, const char *value)
 		wlr_log(WLR_ERROR, "titlebar.height is no longer supported");
 	}
 	if (match_glob(key, "padding.height")) {
-		wlr_log(WLR_ERROR, "padding.height is no longer supported");
+		wlr_log(WLR_INFO, "padding.height is no longer supported");
 	}
 
 	if (match_glob(key, "window.active.border.color")) {