search

w3c / aria-at

Assistive Technology ARIA Experience Assessment
https://aria-at.netlify.app
Other
154 stars 28 forks source link

Refactor Actions Menu Button Example Using element.focus() Test Plan to V2 Test Format #1122

Closed IsaDC closed 2 weeks ago

IsaDC commented 2 months ago

Preview Tests

IsaDC commented 2 months ago

@mcking65 This Pull Request includes the feedback we discussed in today's CG meeting.

IsaDC commented 2 months ago

@mcking65 All done :)

mcking65 commented 4 weeks ago

@IsaDC

There's a bug in the setup script for tests 8. If you first run setup for test 7, the test 8 setup will set focus to the wrong item. If you refresh browser cache, the test 8 script will then work correctly and set focus to the first item.

To reproduce:

  1. Open the test plan in Chrome.
  2. Open the test 7 test window.
  3. Run the setup script for test 7.
  4. Close the test 7 window .
  5. Open the test 8 test window.
  6. Run the test 8 setup.

Expected behavior: Focus is set on action 1.

Actual behavior: Focus is set on action 4.

IsaDC commented 4 weeks ago

@IsaDC

There's a bug in the setup script for tests 8. If you first run setup for test 7, the test 8 setup will set focus to the wrong item. If you refresh browser cache, the test 8 script will then work correctly and set focus to the first item.

To reproduce:

  1. Open the test plan in Chrome.
  2. Open the test 7 test window.
  3. Run the setup script for test 7.
  4. Close the test 7 window .
  5. Open the test 8 test window.
  6. Run the test 8 setup.

Expected behavior: Focus is set on action 1.

Actual behavior: Focus is set on action 4.

Thank you, @mcking65, for reporting this. I haven’t been able to reproduce the issue so far, as both setup scripts seem to be working correctly and setting focus as expected. @jscholes, could you take a look when you have a chance?