Forums Bug Reports Thread

Profile-edit quick-add link chips use undefined CSS variable var(--color-text)

Patrick Bass · Jun 6 · 12 · 1 Locked
[Minor] [Normal Priority] [Bug Fixed] [Always Reproduces]
🚀 OP Jun 6, 2026 5:11pm

Area: Account & identity (audit phase 1) · Surface: /profile/edit (ProfileController@editProfile) · Dimension: law-3-contrast · Severity: minor

The quick-add chips (Mastodon, GitHub, etc.) set an explicit pill background (--color-bg-input) but their text color references a token that does not exist, so the color is whatever the cascade inherits rather than the intended theme foreground. Per the project rule, text inside a hardcoded/explicit-background container must use a valid semantic token, never an undefined one.

Evidence

platform/templates/profile/edit.php L272: style="background:var(--color-bg-input);border:1px solid var(--color-border);...;color:var(--color-text);"
`--color-text` is never defined: `grep -cE '^\s*--color-text:' public/css/style.css` returns 0 and `grep -oE '\-\-color-text\b[^-]'` returns nothing. Only --color-text-primary / --color-text-muted / --color-text-secondary exist. The var() call has no fallback, so the chip text color resolves to `inherit` instead of a theme token, decoupling it from the explicit --color-bg-input background and risking a low-contrast pairing in one theme.

Suggested fix. Change color:var(--color-text) to color:var(--color-text-primary) on platform/templates/profile/edit.php:272.

Filed by the automated tenant-app audit (phase 1) and adversarially evidence-verified. Status: verified. Open — not yet actioned.


Patrick Bass
@mobieus

🚀 Jun 7, 2026 5:31am

Resolved — fixed and deployed. Commit af2b6678f082, shipped dev-first then to all tenants on 2026-06-06.

Changed the quick-add profile-link chip text color from the raw `--color-text` token to the semantic `--color-text-primary` token on the inline style at platform/templates/profile/edit.php:272, so the chip label keeps proper contrast in both themes.

Status: fixed. Thread closed and locked.


Patrick Bass
@mobieus

Log in or register to reply to this thread.