Forums Bug Reports Thread

Share modal preview/close use hardcoded slate rgba instead of semantic tokens

Patrick Bass · Jun 6 · 15 · 1 Locked
[Cosmetic] [Low Priority] [Bug Fixed] [Always Reproduces]
🚀 OP Jun 6, 2026 6:00pm

Area: Files, photos, gallery, ansi (audit p5) · Surface: /photos/album/{id} (Share to feed modal) · Dimension: law3-contrast · Severity: cosmetic

The preview backdrop and close-button hover use a fixed near-black slate tint. On dark themes this near-invisible tint produces almost no separation between the preview area and the card, weakening affordance (the empty-preview placeholder essentially disappears). Per Law 3 these should be semantic tokens that adapt per theme.

Evidence

platform/templates/photos/album.php:1497 `.ps-modal-preview{...background:rgba(15,23,42,.06)...}` and :1495 `.ps-modal-close:hover{background:rgba(15,23,42,.06);...}` — hardcoded slate-900 tints rather than `--color-bg-input` / a hover token. The same block does use semantic tokens elsewhere, so the hardcoded values are an inconsistency.

Suggested fix. Replace rgba(15,23,42,.06) with `var(--color-bg-input)` for the preview background and a themed hover token for .ps-modal-close:hover so the contrast holds across all themes.

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


Patrick Bass
@mobieus

🚀 Jun 7, 2026 5:38am

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

Replaced rgba(15,23,42,.06) with var(--color-bg-input,#f1f3f8) on .ps-modal-preview background and on .ps-modal-close:hover background, so the preview area and close-button hover state stay contrast-correct across light and dark themes.

Status: fixed. Thread closed and locked.


Patrick Bass
@mobieus

Log in or register to reply to this thread.