department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
280 stars 195 forks source link

[Other - Accessibility] Keyboard focus doesn't move as appropriate to workflow. (00.00.3) #73992

Closed shiragoodman closed 5 months ago

shiragoodman commented 5 months ago

General Information

VFS team name

Benefits Non-Disability

VFS product name

Pension 527EZ Digital Form

VFS feature name

Form Reactivation

Point of Contact/Reviewers

Brian DeConinck - @briandeconinck - Accessibility

*For more information on how to interpret this ticket, please refer to the Anatomy of a Staging Review issue ticket guidance on Platform Website.


Platform Issue

No VA.gov Experience Standard for the issue found.

Issue Details

On Gross monthly income, when entering multiple income sources in the list-and-loop, after pressing the Add another income source button keyboard focus doesn't move to the top of the new income source group of inputs.

For a screen reader user, nothing is announced at all --- which may make you question if it worked, try to press enter again, etc. The next tab stop is the Back button (missing the new fields), and a user may erroneously go back, then move forward again.

This was logged as an issue by @vbellisimo in #73299, and it looks like @ToddWebDev may already have a solution in the works. Hopefully this one is an easy one!

This also occurs on:

Link, screenshot or steps to recreate

Steps to reproduce:

  1. Activate a screen reader (NVDA on Windows, VoiceOver on Mac).
  2. Navigate through the form to Gross monthly income.
  3. Enter your first income source.
  4. Press tab to move focus to Add another income source button.
  5. Press enter.
  6. Note that nothing is announced by your screen reader.
  7. Press tab.
  8. Note that keyboard focus moves to the Back button.

VA.gov Experience Standard

Category Number 00, Issue Number 00

Other References

Platform Recommendation

Other list-and-loops in this workflow don't have this issue, so I'm not sure what might be happening here. Please feel free to reach out to me if you need any help testing or validating your solution.


VFS Guidance

aplatt-coforma commented 5 months ago

Moving this to Review/QA since the fix for the related (duplicate) ticket #73299 is already on Staging.

gmaciolek commented 5 months ago

This appears to be resolved by @ToddWebDev 's fix. When I checked it out on staging, after pressing the "Add another income source" button, the keyboard focus goes to the first radio button in the “What type of income?” question in the New income source section. That’s also what’s being announced in VoiceOver on Mac.

ToddWebDev commented 5 months ago

This ticket is resolved with the work completed for internal Issue #73299: https://app.zenhub.com/workspaces/benefits-pension-64e775aaa6b1ca1ed49b2ede/issues/gh/department-of-veterans-affairs/va.gov-team/73299

shiragoodman commented 5 months ago

@aplatt-coforma Are you asking for Governance to help validate that this issue is resolved?

aplatt-coforma commented 5 months ago

@shiragoodman - No, I don't think that is needed to close the ticket right? Per the VFS Guidelines, we just need to validate the fix or get our Product Owner to review.

Since both @gmaciolek and @ToddWebDev have verified it, I think we can close today. Just moved it to review column for our team's visibility.

shiragoodman commented 5 months ago

yup @aplatt-coforma ! that's correct. close when you're ready :)