From 33d74d33f0a0cdd5edd2faa756e80abeaaf14875 Mon Sep 17 00:00:00 2001
From: jackwener <jakevingoo@gmail.com>
Date: Thu, 9 Jul 2026 19:29:59 +0800
Subject: [PATCH] fix(desktop): skills header control alignment + models list
 radius language (round 10)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Maintainer reports, both CDP-measured:
- 技能 header: search field (32px, radius 8) and 添加 CTA (36px, radius 6)
  sat in one row misaligned. Both now 36px / radius-control. Measured
  36/36 + 6px/6px.
- 模型 connection list: interactive row fills (provider trigger hover,
  selected connection) were square full-bleed bands in an app whose
  every other row surface (sidebar nav, session list) rounds its fill.
  Rows now carry radius-control; the seamless-list shell (deliberate
  no-card decision) is unchanged. Measured 0→6px.

Desktop 2285/2285 (exit-code gated).
---
 apps/desktop/src/renderer/styles/module-pages/skills.css | 8 ++++++--
 apps/desktop/src/renderer/styles/settings/models.css     | 8 ++++++++
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/apps/desktop/src/renderer/styles/module-pages/skills.css b/apps/desktop/src/renderer/styles/module-pages/skills.css
index a7ef285678..5f3b61fd1a 100644
--- a/apps/desktop/src/renderer/styles/module-pages/skills.css
+++ b/apps/desktop/src/renderer/styles/module-pages/skills.css
@@ -1,11 +1,15 @@
+/* Round 10: the search field and the 添加 CTA share one header row —
+   they align on the SAME control height (36px, the default button) and
+   the SAME radius family (radius-control). Previously 32px/8px vs
+   36px/6px read as two misaligned controls. */
 .maka-skill-search {
   width: min(280px, 30vw);
-  height: 32px;
+  height: 36px;
   display: inline-flex;
   align-items: center;
   gap: var(--space-1-5);
   border: var(--border-width-hairline) solid var(--border);
-  border-radius: var(--radius-surface);
+  border-radius: var(--radius-control);
   background: var(--background);
   color: var(--muted-foreground);
   padding: 0 var(--space-2-5);
diff --git a/apps/desktop/src/renderer/styles/settings/models.css b/apps/desktop/src/renderer/styles/settings/models.css
index 641d9d19f2..28ba8498c5 100644
--- a/apps/desktop/src/renderer/styles/settings/models.css
+++ b/apps/desktop/src/renderer/styles/settings/models.css
@@ -34,6 +34,10 @@
   transition: background-color var(--duration-quick) var(--ease-out-strong);
 }
 
+/* Round 10: interactive fills in the seamless list are ROUNDED like every
+   other row surface in the app (sidebar nav, session list) — square
+   full-bleed bands broke the radius language. */
+.enabledProviderTrigger { border-radius: var(--radius-control); }
 .enabledProviderTrigger:hover { background: var(--state-hover-bg); }
 
 .enabledProviderName {
@@ -108,6 +112,10 @@
   border-top: var(--border-width-hairline) solid oklch(from var(--foreground) l c h / 0.07);
 }
 
+.enabledConnRow {
+  border-radius: var(--radius-control);
+}
+
 .enabledConnRow[data-default="true"] {
   background: var(--state-selected-bg);
 }