From 583d73c53cb78812b37458454120d4f37206f43d Mon Sep 17 00:00:00 2001
From: Breadway <plasticbread849@gmail.com>
Date: Tue, 25 Aug 2026 21:14:57 +0800
Subject: [PATCH] liquid-motion workspace pills: match demo radius and height
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

.workspace-trail/.workspace-btn hardcoded border-radius: 12px and
min-height: 28px — neither matches this theme's own radius_sm token
(9px) nor the 01-liquid-motion.html demo's .ws-btn/.trail rules (26px
tall, 9px radius). radius_sm happens to be an exact match for the
demo's 9px here. Reported: "the pills on liquid motion just look
off".

Not visually verified via capture — the isolated headless-Sway
harness has no Hyprland IPC, so workspace buttons render empty there
regardless of theme (pre-existing harness limit, unrelated to this
change). Verified by reading the CSS against 01-liquid-motion.html's
<style> block instead, and by `cargo build`/clippy/test passing clean.
---
 src/theme.rs | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/src/theme.rs b/src/theme.rs
index 8dba26c..c7c5cb2 100644
--- a/src/theme.rs
+++ b/src/theme.rs
@@ -127,12 +127,20 @@ fn load_css() -> String {
     // (see `App::rebuild_buttons`) — the trail's own `.workspace-trail`
     // pill CSS is therefore irrelevant for them (it's never made visible).
     let workspace_css = match theme.modules().workspaces.style {
+        // Radius and height were hardcoded (12px, 28px) instead of reading
+        // from this theme's own tokens/demo — 12px is neither radius_sm
+        // (9px) nor any other token this theme defines, and the demo's
+        // `.ws-btn`/`.trail` both draw a 26px-tall, 9px-radius pill (not
+        // 28px/12px). radius_sm happens to be an exact match for the
+        // demo's 9px here, unlike glass-workbench's Pill style below where
+        // it's also used but for a different, already-correct reason.
+        // Reported: "the pills on liquid motion just look off".
         bread_theme::shell::WorkspaceStyle::Trail => format!(
             ".workspace-trail {{ background-image: linear-gradient(90deg, @accent, @teal);\
-                 background-color: @accent; border-radius: 12px; }}\
+                 background-color: @accent; border-radius: {radius_sm}; }}\
              .workspace-btn {{ background: transparent; opacity: 0.36; color: @on-bg;\
-                 border-radius: 12px; border: none; outline: none; box-shadow: none;\
-                 min-width: 28px; min-height: 28px; margin: 0; padding: 0 7px;\
+                 border-radius: {radius_sm}; border: none; outline: none; box-shadow: none;\
+                 min-width: 28px; min-height: 26px; margin: 0; padding: 0 7px;\
                  font-size: 22px; font-weight: bold;\
                  transition: opacity 0.22s {spring_settle},\
                      background-color 0.22s {spring_settle}; }}\