- πΊπΈUnited States jldust
The most recent patches were failing so I rerolled the patch presented in #61 π Convey AJAX progress messages to assistive technology. Needs work against the 9.5.x branch and was able to get it to apply cleanly. Hopefully we can get some additional accessibility review around the effectiveness of these changes.
I would be happy to help with additional test cases & further edits to get this moving along. Should we also start planning for this against the 10.1.x branch as well to be proactive?
- last update
over 1 year ago Custom Commands Failed - πΊπΈUnited States jldust
Looks like the spelling of "screenreader" is causing a build error so I have adjusted the two places called out to "screen reader".
- last update
over 1 year ago Custom Commands Failed - πΊπΈUnited States jldust
Updated build command was missed during reroll, this should account for those changes.
- last update
over 1 year ago Custom Commands Failed - πΊπΈUnited States jldust
Fixed spelling regression after updating ES6
- last update
over 1 year ago 30,331 pass, 1 fail - Status changed to Needs review
over 1 year ago 2:02pm 15 May 2023 - last update
over 1 year ago 30,334 pass - πΊπΈUnited States jldust
It looks like passing more than one argument in
WebAssert::responseContains
was deprecated in Drupal 9.1. I've removed the secondary argument as it is also listed in the comments directly above the line. - Status changed to Needs work
over 1 year ago 6:24pm 15 May 2023 - πΊπΈUnited States smustgrave
Can the issue summary be updated. Number of "tbd" in there.
Looking at the test coverage can we also add a test for the message in the announce. Make sure that is getting the right message.
- last update
about 1 year ago 30,357 pass, 2 fail - @finnsky opened merge request.
- last update
about 1 year ago 30,357 pass, 2 fail - last update
about 1 year ago Custom Commands Failed - last update
about 1 year ago Custom Commands Failed - last update
about 1 year ago 30,428 pass