touchpoints icon indicating copy to clipboard operation
touchpoints copied to clipboard

WIP focus setting for modal open and close

Open iamjolly opened this issue 3 years ago • 5 comments

For @ryanwoldatwork and @iamjolly to pair on.

Relates to #480

Things to do:

  • [x] Set focus on modal div when opened
  • [x] Set focus back to touchpoints button when modal is closed
  • [x] Confirm listener for esc keypress to run the handleDialogClose function.
  • [ ] Trap focus in modal when opened - see Using JavaScript to trap focus in an element for more info
  • [ ] Test with keyboard only and with screen reader to confirm everything works as expected.

Note: This is a vanilla JS approach to adding accessibility to the modal. A next effort should involve looking at ways for touchpoints to use the USWDS modal component per #481.

iamjolly avatar Feb 03 '22 20:02 iamjolly

thanks @iamjolly!!

ryanwoldatwork avatar Feb 03 '22 21:02 ryanwoldatwork

paired today and check a few boxes here

ryanwoldatwork avatar Feb 08 '22 21:02 ryanwoldatwork

Hey @iamjolly @ryanwoldatwork I'm working on a project that would implement Touchpoints on childcare.gov. Wanted to check in on the status of this update. Thanks in advance!

cannandev avatar Mar 02 '22 23:03 cannandev

hi @cannandev - the path forward is pretty clear. Hoping to knock this out late next week.

ryanwoldatwork avatar Mar 03 '22 18:03 ryanwoldatwork

@iamjolly , @ryanwoldatwork : Could you please provide a status update on this? Thanks!

mharvey avatar Nov 16 '22 21:11 mharvey