Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Just looking at my site through http://wave.webaim.org/toolbar
Came obvious that there are accessibility problems with the Masquerade form (see image attached).
All forms need to have a label with them. These don't need to be visible, but they do need to be there to provide semantics.
Comment | File | Size | Author |
---|---|---|---|
#3 | 1502804-accessibility-3.patch | 3.12 KB | andypost |
#3 | masq_acessibility.jpg | 7.61 KB | andypost |
#1 | 1502804-accessibility-1.patch | 926 bytes | andypost |
Masquerade-accessibility-nolabel.png | 13.76 KB | mgifford |
Comments
Comment #1
andypostDoes this patch make a sense? Maybe title should be shorter?
Comment #2
mgiffordThat would work, but would be a bit of a duplication. as the screen reader user would hear the title twice. I do wonder why you added
<div class="description"></div>
. I'm also not sure why you couldn't just remove$markup_value
completely & use the title (without the invisible title of course).Comment #3
andypost@mgifford We can't use this long string as title because it related to both input and submit. This ugly fix just tried to implement (mimic) standard element description... but this anyway breaks a form flow... so
here's a new patch, changes
- added title Username, probably we should hide it with
'#title_display' => 'invisible'
- reordered a code and added some comments
-also changed a way to display title for quick switches to more accessible
I'd like to get review of @deekayen on this because this changes the form a bit
Comment #4
mgiffordPatch still seems to work well when I ran it in http://simplytest.me
Comment #6
deekayen CreditAttribution: deekayen commentedTraditionally, if you're going to include a description, it's associated with an input field. If you're going to break from that model, I'd think you'd want the instructions to precede the form, not append to it, then again, I'm not blind. If we're really going to consider the impacts of blind people, the Go description on the submit button isn't very helpful either.
Comment #7
andyposton a one hand this needs better accessible attributes and render
Anyway this description should use same rules as inputs does
on a second - this sould be done with proper container wrapper as 8.x-2.x makes.
So description could be attached to container
Comment #8
deekayen CreditAttribution: deekayen commentedI'm not highly opinionated on this issue and don't have any background in blind people stuff, nor am I really interested in getting one.
Comment #9
Melissamcewen CreditAttribution: Melissamcewen commented"blind people stuff"
Err, let's just call it Accessibility, since it applies to a wide range of people who use screen readers and other accessibility devices, not just "blind people." Also it is of wide interest because a lot of us work in fields like government or education where we are legally required to produce accessible websites. But that can be a bit hard to work out- should a dev do the bare minimum which is to make it readable in a screen reader and adhere to basic web best practices, or should they be doing usability testing with a screen reader? Probably the former is best for modules like this.
Comment #10
deekayen CreditAttribution: deekayen commentedYeah, I get it. I don't work for the government and I've turned down jobs where I found their portfolio had govt work I might have to participate in. There are too many government departments that have no legitimate constitutional existence and I won't participate willingly in their success.
#8 is simply me letting Andrey take lead on this issue. I don't really care about accessibility, nor would I attempt to maintain its compatibility in the future, but this module has quickly grown to be completely out of my control anyway. Andrey, Mike, and Daniel are good devs and I don't feel the need to clash on this issue.