Skip to content

fix: theme color, accessibility, and cleanup improvements#271

Merged
tomasz-tomczyk merged 2 commits intomainfrom
audit/frontend-fixes
Apr 14, 2026
Merged

fix: theme color, accessibility, and cleanup improvements#271
tomasz-tomczyk merged 2 commits intomainfrom
audit/frontend-fixes

Conversation

@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Summary

  • Replace hardcoded #34d399 with var(--green) in .live-thread-badge
  • Extract buildContentClasses helper to deduplicate render paths
  • Add ArrowLeft/ArrowRight keyboard nav for settings tabs (ARIA tabs pattern)
  • Add aria-label feedback and live region for copy button screen reader announcements
  • Replace window.dismissToast global with event delegation on toast container
  • Show share button as disabled with tooltip in git mode instead of hiding it

Test plan

  • Theme color uses CSS variable, verified --green defined in all theme contexts
  • ARIA tab pattern verified (role, aria-selected, aria-controls, keyboard nav)
  • Copy announcements use role="status" live region
  • No new globals introduced (net reduction: removed window.dismissToast)

🤖 Generated with Claude Code

tomasz-tomczyk and others added 2 commits April 14, 2026 21:57
- Replace hardcoded #34d399 with var(--green) in .live-thread-badge
- Extract buildContentClasses helper to deduplicate render paths
- Add ArrowLeft/ArrowRight keyboard nav for settings tabs (ARIA pattern)
- Add aria-label feedback and live region for copy button announcements
- Replace window.dismissToast global with event delegation
- Show share button as disabled with tooltip in git mode

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The share button is now visible but disabled in git mode instead of
hidden, so the E2E test needs to assert toBeDisabled() + tooltip.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tomasz-tomczyk tomasz-tomczyk merged commit f46dbd0 into main Apr 14, 2026
3 checks passed
@tomasz-tomczyk tomasz-tomczyk deleted the audit/frontend-fixes branch April 14, 2026 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant