- π¨π¦Canada JayDarnell Guelph, Ontario
With this still being a problem in the 2.x branch I'm going to update the issue appropriately, create an issue fork, and reroll adriancruz's patch from #2 for the latest 2.x release.
- Assigned to JayDarnell
- π¨π¦Canada JayDarnell Guelph, Ontario
Also updating this to a major bug due to the fact the module as written will not pass accessibility tests.
- last update
almost 2 years ago 19 pass, 4 fail - @jaydarnell opened merge request.
- π¨π¦Canada JayDarnell Guelph, Ontario
I can update the issue fork to add a span around the subset of the label containing the equation in order to satisfy the failed test but wanted to get maintainer feedback first.
- πΊπΈUnited States mrdrewkeller
Re-rolled this patch for the 2.x branch
- Issue was unassigned.
- Status changed to Needs review
about 1 month ago 1:18pm 24 June 2025 - π«π·France simon georges Rouen, France
I can confirm that we just had this issue reported to us during a mandatory accessibility audit ;-)
Thanks for the patch!
- π«π·France simon georges Rouen, France
As a side note, considering this makes the captcha inaccessible, this prevents people from using forms, which may be quite important.
There I'm raising the priority of the issue.
- First commit to issue fork.
- π©πͺGermany Anybody Porta Westfalica
Thanks, this also makes the captcha more accessible for bots, but I agree accessibility for humans is more important and in general this captcha type isn't highly secure.
RTBC! Let's merge this! :)
-
anybody β
committed 48be3794 on 2.x authored by
jaydarnell β
Issue #3221197 by jaydarnell, adriancruz, anybody, mrdrewkeller, simon...
-
anybody β
committed 48be3794 on 2.x authored by
jaydarnell β
- π©πͺGermany Anybody Porta Westfalica
Sorry needs to be reopened to fix the test, I didn't see that -.-
- π©πͺGermany Grevil
@anybody, let's create a follow-up issue for that.
Created it here: π Fix test regression causes through issue 3221197 Active .