From e372640f0f6415cebdcd5788b679282b3ef739d9 Mon Sep 17 00:00:00 2001 From: dev Date: Thu, 21 May 2026 00:11:56 +0200 Subject: [PATCH] accordion: tone down open-trigger highlight (no pink text) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The open accordion trigger was rendering in `--color-primary-text` (pink/magenta in dark theme), which read as a loud accent on the heading — visually noisy and inconsistent with shadcn-svelte / Radix / Ark UI, which all keep the trigger text in the regular content tone and let the rotated chevron + revealed content signal "open". Recipe tokens updated: - `accordion.trigger-color-open`: `--color-primary-text` → `--color-content-primary` (no recolor on open). - `accordion.indicator-color-open`: `--color-primary-text` → `--color-content-primary` (subtle emphasis without the loud accent). `generated/base.css` regenerated. Verified in browser: open trigger heading now matches the closed triggers' tone; the chevron rotates upward and the content reveals as the only open affordances. Co-Authored-By: Claude Opus 4.7 (1M context) --- src/uix/eidos/generated/base.css | 4 ++-- src/uix/eidos/lib/recipes/base.ts | 10 ++++++++-- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/src/uix/eidos/generated/base.css b/src/uix/eidos/generated/base.css index e74c0d997..9f258a89b 100644 --- a/src/uix/eidos/generated/base.css +++ b/src/uix/eidos/generated/base.css @@ -315,7 +315,7 @@ --accordion-trigger-font-weight: var(--font-weight-medium); --accordion-trigger-line-height: var(--leading-ui); --accordion-trigger-color: var(--color-content-primary); - --accordion-trigger-color-open: var(--color-primary-text); + --accordion-trigger-color-open: var(--color-content-primary); --accordion-trigger-color-disabled: var(--color-content-disabled); --accordion-trigger-hover-bg: var(--color-surface-raised); --accordion-trigger-hover-color: var(--color-content-primary); @@ -344,7 +344,7 @@ --accordion-indicator-size-md: var(--icon-size-sm); --accordion-indicator-size-lg: var(--icon-size-md); --accordion-indicator-color: var(--color-content-secondary); - --accordion-indicator-color-open: var(--color-primary-text); + --accordion-indicator-color-open: var(--color-content-primary); --accordion-indicator-rotation: 180deg; --accordion-indicator-transition-duration: var(--duration-fast); --accordion-indicator-transition-ease: var(--ease-default); diff --git a/src/uix/eidos/lib/recipes/base.ts b/src/uix/eidos/lib/recipes/base.ts index efbc6fe72..2d3ce243e 100644 --- a/src/uix/eidos/lib/recipes/base.ts +++ b/src/uix/eidos/lib/recipes/base.ts @@ -39,7 +39,11 @@ export const THEME_BASE_RECIPE_TOKENS = { 'trigger-font-weight': 'var(--font-weight-medium)', 'trigger-line-height': 'var(--leading-ui)', 'trigger-color': 'var(--color-content-primary)', - 'trigger-color-open': 'var(--color-primary-text)', + // Open state: keep the trigger text in the primary content tone. + // The chevron rotation + revealed content already signal openness; + // recoloring the heading adds noise without information (matches + // shadcn/Radix/Ark). + 'trigger-color-open': 'var(--color-content-primary)', 'trigger-color-disabled': 'var(--color-content-disabled)', 'trigger-hover-bg': 'var(--color-surface-raised)', 'trigger-hover-color': 'var(--color-content-primary)', @@ -68,7 +72,9 @@ export const THEME_BASE_RECIPE_TOKENS = { 'indicator-size-md': 'var(--icon-size-sm)', 'indicator-size-lg': 'var(--icon-size-md)', 'indicator-color': 'var(--color-content-secondary)', - 'indicator-color-open': 'var(--color-primary-text)', + // Indicator turns to the primary content tone when open — a subtle + // emphasis without the loud accent recolor. + 'indicator-color-open': 'var(--color-content-primary)', 'indicator-rotation': '180deg', 'indicator-transition-duration': 'var(--duration-fast)', 'indicator-transition-ease': 'var(--ease-default)'