From 2746d1408c2a7dedd1b6ecf9a22cc8c4381b8595 Mon Sep 17 00:00:00 2001 From: Keysat Date: Sat, 20 Jun 2026 15:46:44 -0500 Subject: [PATCH] Dim disabled secondary sheet buttons (review polish) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Holistic review nit: .sheet-remove (Reject/Delete/Remove) and the bell's Back button accepted `disabled` but had no dimmed state, unlike .sheet-submit — so a tap during an in-flight write gave no feedback. Add a 0.5-opacity disabled rule to both. CSS-only; still v0.1.0:102 (not yet built/deployed). --- frontend/index.html | 3 +++ 1 file changed, 3 insertions(+) diff --git a/frontend/index.html b/frontend/index.html index 14dcd56..2bafe64 100644 --- a/frontend/index.html +++ b/frontend/index.html @@ -2611,6 +2611,7 @@ .bell-meta { font-family: 'IBM Plex Mono', monospace; font-size: 12px; color: var(--text-subtle); margin: -4px 0 10px; } .bell-summary { font-size: 13px; color: var(--text-secondary); line-height: 1.45; margin: 0 0 14px; padding: 10px 12px; background: var(--bg-input); border: 1px solid var(--border); border-radius: 8px; } .bell-back { width: 100%; margin-top: 10px; background: transparent; border: none; color: var(--accent-light); font-size: 14px; font-family: inherit; cursor: pointer; padding: 8px; } + .bell-back:disabled { opacity: 0.5; cursor: default; } .quicklog-hint { font-size: 13px; color: var(--text-subtle); line-height: 1.5; margin: 0 0 12px; } .quicklog-pool { display: flex; flex-direction: column; gap: 8px; margin-top: 12px; } .quicklog-empty { font-size: 13px; color: var(--text-subtle); padding: 16px 4px; } @@ -2708,6 +2709,8 @@ background: transparent; border: 1px solid var(--danger-soft); color: var(--danger-soft); border-radius: var(--mobile-control-radius); font-size: 14px; font-family: inherit; cursor: pointer; } + /* Match .sheet-submit's disabled affordance on the secondary actions (Reject, Back, Delete). */ + .sheet-remove:disabled { opacity: 0.5; cursor: default; } .dedup-box { background: var(--bg-input); border: 1px solid var(--border); border-radius: var(--mobile-control-radius); padding: 8px 12px; margin: -4px 0 14px; } .dedup-box-title { font-size: 12px; color: var(--due-soon, #e0b341); margin-bottom: 4px; } .dedup-match { font-size: 13px; color: var(--text-secondary); padding: 3px 0; }