fix(frontend): sprint 4 post-code-review — btn-ink uses slab token + unify New CTA label

- 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>
This commit is contained in:
Knacky
2026-05-27 20:40:06 +02:00
parent 988de841e5
commit e99286ef8e
3 changed files with 3 additions and 4 deletions

View File

@@ -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
}

View File

@@ -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;

View File

@@ -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)