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

Apply WCAG standards to log in screen #1722

Open
10 of 15 tasks
SLDonnelly opened this issue Nov 18, 2024 · 2 comments · May be fixed by #1742
Open
10 of 15 tasks

Apply WCAG standards to log in screen #1722

SLDonnelly opened this issue Nov 18, 2024 · 2 comments · May be fixed by #1742
Assignees
Labels
front-end front-end related task

Comments

@SLDonnelly
Copy link
Collaborator

SLDonnelly commented Nov 18, 2024

As a SPAR User I want SPAR to follow WCAG standards So That I can use assistive technologies to help me complete my work

Acceptance Criteria

For the SPAR log in screen
image

  • Semantic HTML structure
  • Ensure all images have alternative text (for the photo on the log in screen use: "Picture of spruce tree branches with needles and cones".
  • Accessible forms
  • Keyboard navigation
  • ARIA (Accessible Rich Internet Applications) roles and attributes
  • Focus management
  • Check that contrast is above 4.5:1
  • Tested - Use accessibility tools and screen readers during development.

Additional Context

Definition of Done

  • Ready to Demo in Sprint Review
  • Does what I have made have appropriate test coverage?
  • Documentation and/or scientific documentation exists and can be found
  • Peer Reviewed by 2 people on the team
  • Manual testing of all PRs in Dev and Prod
  • QA from UX team member
  • Merged
@ngunner15
Copy link
Collaborator

To Do before working on WCAG tickets:

  • Read up on the documentation for various HTML element's WCAG requirements.
  • Example scenario

What is the correct tag for a text? Is it <h1>, <p> or for styling purposes it should be <div>. If we use <div> and it should behave as a heading or title then we have to add aria labels to show <div> as a heading.
eg <div role="heading" aria-level="2">Title Text Here</div>

  • Find a screen reader testing tool preferably one that can be integrated with a browser like a Chrome extension, for eg WAVE

@kevinginley
Copy link
Collaborator

@ngunner15 I did a manual review using WAVE and the page is meeting the outlined criteria. Great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment