Skip to content

fix: a11y violations in toast/button/screen stories (#163) - #168

Open
luandro wants to merge 3 commits into
mainfrom
agent/comapeo-cloud-app/issue-163
Open

fix: a11y violations in toast/button/screen stories (#163)#168
luandro wants to merge 3 commits into
mainfrom
agent/comapeo-cloud-app/issue-163

Conversation

@luandro

@luandro luandro commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Fixes a11y violations in Storybook stories:

  • Wrap toast stories in landmark
  • Use navy #04145C for toast trigger buttons (color-contrast fix)
  • Add bg-navy border-navy to Button Primary story (color-contrast fix)
  • Wrap DataScreen story in ToastProvider + (fixes useToast crash)
  • Add decorator to SettingsScreen story (a11y landmark)

All 135 storybook tests pass, coverage at 87.66%.

- Wrap toast stories in <main> landmark for a11y
- Use navy #04145C for toast trigger buttons (color-contrast fix)
- Add bg-navy border-navy to Button Primary story (color-contrast fix)
- Wrap DataScreen story in ToastProvider + <main> (fixes useToast crash)
- Add <main> decorator to SettingsScreen story (a11y landmark)

Resolves #163
@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Preview deployment ready: https://agent-comapeo-cloud-app-issu-97kh.comapeo-cloud-app.pages.dev

Commit: 86d878c

…anges

The a11y contrast fixes in the toast and Data screen stories intentionally
change their rendered appearance, so the mobile Storybook screenshot
baseline diverged from CI and the visual-regression-check failed.

Regenerate the affected baseline images from CI's own renders (the 8 mobile
stories that exceed the 1% pixel threshold) so the check matches:
- components-toast: error/info/stacked/success/with-description
- screens-data: error-state/with-project-and-data/with-project-desktop

No threshold changes and no test skips — the visual changes are real and
the baseline is brought back in sync with them.

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant