Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Contact Form #245

Open
buddy-harlow opened this issue Jul 26, 2024 · 0 comments · May be fixed by #384
Open

Contact Form #245

buddy-harlow opened this issue Jul 26, 2024 · 0 comments · May be fixed by #384
Labels
a11y Affects accessibility.

Comments

@buddy-harlow
Copy link
Collaborator

buddy-harlow commented Jul 26, 2024

Location: https://www.bixal.com/

Design

  • Placeholder text is acting as form label.
  • There are no instructions stating "All fields marked with * are required"

Error identification

  • Only being handled by HTML5.
  • When I leave a required field empty and press Submit, the screen reader announces "Please fill out this field".

Suggestion

  • Recommend that we have permanent form labels and not rely on placeholder text
  • Ensure we include required fields are marked with asterisks.
  • When I make an error, I expect error identification to specifically identify which field(s) I need to fix.
  • If I make an error, I expect error suggestions to let me know how I can correct my mistake. (ie. If I my email is formatted incorrectly and missing an @ or .com)

Resource: Deque Blog - The Anatomy of Accessible Forms: Error Messages

@buddy-harlow buddy-harlow added the a11y Affects accessibility. label Jul 26, 2024
mejiaj added a commit that referenced this issue Mar 10, 2025
In storybook, it'll try to run on all pages and cause an error when header nav is missing.
mejiaj added a commit that referenced this issue Mar 10, 2025
- Add contact class for background image
- Add CAPTCHA placeholder text
- Pass secondary button variant to reset button
mejiaj added a commit that referenced this issue Mar 10, 2025
These rules are unnecessary now that we have a reusable grid.
mejiaj added a commit that referenced this issue Mar 10, 2025
Otherwise when content is long one column can extend beyond defined width.
@mejiaj mejiaj linked a pull request Mar 10, 2025 that will close this issue
mejiaj added a commit that referenced this issue Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Affects accessibility.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant