From e14fad17096a80e35c4162c59b219085ca85e1a6 Mon Sep 17 00:00:00 2001
From: amosblomqvist <amosblomqvist@gmail.com>
Date: Fri, 18 Sep 2026 08:57:10 +0300
Subject: [PATCH] =?UTF-8?q?fix:=20model=20menu=20highlight=20matches=20qui?=
 =?UTF-8?q?z=20rows=20=E2=80=94=20filled=20accent=20tint,=20no=20clipped?=
 =?UTF-8?q?=20ring?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The walked row's ring-1 is a box-shadow painted OUTSIDE the border box;
the list's overflow-y-auto makes overflow-x clip at the padding box too
(CSS: a non-visible axis forces the other to auto), so the 1px halo was
shaved on both sides. Replaced with the quiz rows' bg-accent/40 filled
tint and added px-1 to the list so the highlight never touches the
edges (owner round 5).
---
 components/model-picker.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/components/model-picker.tsx b/components/model-picker.tsx
index a02f50f..852e533 100644
--- a/components/model-picker.tsx
+++ b/components/model-picker.tsx
@@ -312,7 +312,7 @@ export function ModelControl({
 					{catalog && results.length === 0 && (
 						<div className="px-2 py-3 text-center text-muted-foreground">No models match.</div>
 					)}
-					<ul ref={listRef} data-testid="model-list" className="flex max-h-72 flex-col overflow-y-auto">
+					<ul ref={listRef} data-testid="model-list" className="flex max-h-72 flex-col overflow-y-auto px-1">
 						{results.map((m, i) => (
 							<li key={`${m.provider}/${m.id}`}>
 								<button
@@ -321,7 +321,7 @@ export function ModelControl({
 									data-idx={i}
 									onClick={() => pickModel(m)}
 									className={`flex w-full flex-col gap-0.5 rounded-lg px-2 py-1.5 text-left outline-none transition-colors hover:bg-muted ${
-										i === menuIdx ? "bg-muted ring-1 ring-border" : ""
+										i === menuIdx ? "bg-accent/40" : ""
 									}${m.id === effectiveId && i !== menuIdx ? " bg-muted/60" : ""}`}
 								>
 									<span className="flex items-center justify-between gap-2">