From fb0c7a29c68b2862ff9920bf53863a42a952b85d Mon Sep 17 00:00:00 2001
From: tokyo4j <hrak1529@gmail.com>
Date: Thu, 3 Oct 2024 09:23:50 +0900
Subject: [PATCH] theme: draw hover overlay in the correct scale

The hover icon buffer on which the hover overlay is draw can be larger
than the button size when a large non-hover icon is provided by user. In
this case, we should scale up all the coorinates that are used to draw
the hover overlay. Before this commit, corner radius and padding.width
are not scaled correctly.

This commit fixes it by scaling the cairo context with cairo_scale()
when drawing the hover overlay.
---
 src/theme.c | 40 ++++++++++++++++++++++------------------
 1 file changed, 22 insertions(+), 18 deletions(-)

diff --git a/src/theme.c b/src/theme.c
index b3c864d8..f3b328b0 100644
--- a/src/theme.c
+++ b/src/theme.c
@@ -135,34 +135,37 @@ create_hover_fallback(struct theme *theme, const char *icon_name,
 	int width = theme->window_button_width;
 	int height = theme->title_height;
 
-	if (width && height) {
-		/*
-		 * Proportionately increase size of hover_buffer if the
-		 * non-hover 'donor' buffer is larger than the allocated space.
-		 * It will get scaled down again by wlroots when rendered and as
-		 * required by the current output scale.
-		 *
-		 * This ensures that icons > width or > height keep their aspect
-		 * ratio and are rendered the same as without the hover overlay.
-		 */
-		double scale = MAX((double)icon_width / width,
-				(double)icon_height / height);
-		if (scale > 1.0f) {
-			width = (double)width * scale;
-			height = (double)height * scale;
-		}
+	/*
+	 * Proportionately increase size of hover_buffer if the non-hover
+	 * 'donor' buffer is larger than the allocated space. It will get
+	 * scaled down again by wlroots when rendered and as required by the
+	 * current output scale.
+	 *
+	 * This ensures that icons > width or > height keep their aspect ratio
+	 * and are rendered the same as without the hover overlay.
+	 */
+	double scale = (width && height) ?
+		MAX((double)icon_width / width, (double)icon_height / height) : 1.0;
+	if (scale < 1.0) {
+		scale = 1.0;
 	}
+	int buffer_width = (double)width * scale;
+	int buffer_height = (double)height * scale;
 
-	*hover_buffer = buffer_create_cairo(width, height, 1.0, true);
+	*hover_buffer = buffer_create_cairo(buffer_width, buffer_height, 1.0, true);
 
 	cairo_t *cairo = (*hover_buffer)->cairo;
 	cairo_surface_t *surf = cairo_get_target(cairo);
 
 	/* Background */
 	cairo_set_source_surface(cairo, icon.surface,
-		(width - icon_width) / 2, (height - icon_height) / 2);
+		(buffer_width - icon_width) / 2, (buffer_height - icon_height) / 2);
 	cairo_paint(cairo);
 
+	/* Switch from buffer scale to theme scale */
+	cairo_save(cairo);
+	cairo_scale(cairo, scale, scale);
+
 	/* Overlay (pre-multiplied alpha) */
 	float overlay_color[4] = { 0.15f, 0.15f, 0.15f, 0.3f};
 	int radius = MIN(width, height) / 2;
@@ -212,6 +215,7 @@ create_hover_fallback(struct theme *theme, const char *icon_name,
 		break;
 	}
 	cairo_surface_flush(surf);
+	cairo_restore(cairo);
 
 	if (icon.is_duplicate) {
 		cairo_surface_destroy(icon.surface);