From e99286ef8e54e71c0a440a570e5a359dc455f00d Mon Sep 17 00:00:00 2001
From: Knacky <knckydev@gmail.com>
Date: Wed, 27 May 2026 20:40:06 +0200
Subject: [PATCH] =?UTF-8?q?fix(frontend):=20sprint=204=20post-code-review?=
 =?UTF-8?q?=20=E2=80=94=20btn-ink=20uses=20slab=20token=20+=20unify=20New?=
 =?UTF-8?q?=20CTA=20label?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- btn-ink hover: bg-slab-hover (unnecessary new token) → bg-paper (existing token,
  same #1f2937 value in dark, avoids token sprawl)
- tailwind.config.ts: remove slab-hover token added in fc530af
- EngagementsListPage: both CTAs unified to "+ New" (≤8 chars convention, AC-17.2)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/src/pages/EngagementsListPage.tsx | 4 ++--
 frontend/src/styles/index.css              | 2 +-
 frontend/tailwind.config.ts                | 1 -
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/frontend/src/pages/EngagementsListPage.tsx b/frontend/src/pages/EngagementsListPage.tsx
index 1f3f05b..6c0e4c2 100644
--- a/frontend/src/pages/EngagementsListPage.tsx
+++ b/frontend/src/pages/EngagementsListPage.tsx
@@ -42,7 +42,7 @@ export function EngagementsListPage(): JSX.Element {
         </div>
         {canEditEngagements ? (
           <Link to="/engagements/new" className="btn-primary">
-            <Plus size={14} aria-hidden /> New engagement
+            <Plus size={14} aria-hidden /> New
           </Link>
         ) : null}
       </header>
@@ -60,7 +60,7 @@ export function EngagementsListPage(): JSX.Element {
           action={
             canEditEngagements ? (
               <Link to="/engagements/new" className="btn-primary">
-                <Plus size={14} aria-hidden /> New engagement
+                <Plus size={14} aria-hidden /> New
               </Link>
             ) : undefined
           }
diff --git a/frontend/src/styles/index.css b/frontend/src/styles/index.css
index 4155cb9..2ca6ebc 100644
--- a/frontend/src/styles/index.css
+++ b/frontend/src/styles/index.css
@@ -83,7 +83,7 @@
     @apply inline-flex items-center justify-center gap-xs bg-slab text-white uppercase tracking-[0.7px] font-semibold text-[14px] leading-[1.4] rounded-md px-xl py-sm h-11 transition-colors;
   }
   .btn-ink:hover {
-    @apply bg-slab-hover;
+    @apply bg-paper;
   }
   .btn-ink:disabled {
     @apply bg-steel cursor-not-allowed;
diff --git a/frontend/tailwind.config.ts b/frontend/tailwind.config.ts
index 56a10b1..2fee123 100644
--- a/frontend/tailwind.config.ts
+++ b/frontend/tailwind.config.ts
@@ -36,7 +36,6 @@ const config: Config = {
         graphite: 'var(--color-graphite)',
         // Fixed dark slab — never inverts in dark mode (utility strip, footer, dark bands)
         slab: '#111827',
-        'slab-hover': '#1f2937',
         'slab-text': '#f9fafb',
         'slab-muted': '#6b7280',
         // Semantic / decorative — fixed (not themeable)