Theme QA: verify tokenized light/dark modes

[AndreRobitaille]

Summary

• Perform theme QA to ensure the tokenized light/dark theme is coherent, accessible, and free of regressions.
• Focus on readability, focus/hover/visited states, and cross-browser sanity checks without overdesign.

Goal

Catch visual regressions and ensure a consistent, readable UI without overdesign.

Checklist

• Contrast:
  • Verify primary text on background and on surfaces is readable.
  • Verify muted text is still readable (not too faint).
  • Verify links are clearly identifiable in both modes.
• Focus states:
  • Keyboard focus ring visible on links/buttons/inputs.
• Hover states:
  • Meeting rows hover uses --c-surface-2 and is visible but subtle.
• Visited links:
  • Visited links have a distinct but restrained style (avoid bright purple).
• Status pills:
  • All statuses readable in both modes.
• Cards:
  • Accent treatments visible but not loud.
• Filters/search inputs:
  • Input borders and placeholder text readable in both modes.
• Cross-browser quick pass (at least Chrome/Firefox):
  • No unreadable text or invisible borders.

Acceptance Criteria

• A short QA report (notes or checklist marked complete) is added to the issue.
• Any discovered regressions are fixed or spun into follow-up issues.

[AndreRobitaille]

Theme QA completed. Audit findings and fixes:

Fixed:

• Dark mode --color-text-muted contrast: #6c6c70 → #909094 (was 3.3-3.6:1, now ~4.7-5.4:1, WCAG AA compliant)
• Added global :focus-visible rule for all interactive elements (2px solid primary, 2px offset)
• Upgraded form input focus from :focus to :focus-visible
• Footer hardcoded #ffffff → var(--color-header-text) (2 instances)
• Defined --color-surface-hover token in both light/dark modes (was undefined, relying on fallback)
• Removed fallback from .topic-headline-item:hover now that token is defined

Checked and left as-is:

• Header rgba(255, 255, 255, 0.1) — correct for permanently dark header surface
• Modal overlay rgba(0, 0, 0, 0.5) — universal, theme-independent
• Light mode contrast — all combinations pass WCAG AA/AAA
• Placeholder text — uses --color-text-muted token (now contrast-fixed in dark mode)
• Status pills — readable in both modes via semantic color tokens

Not added (intentional):

• :visited link styles — topic cards are the primary nav pattern, not text links. Visited styling would add visual noise without helping scanability.