https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934

--- Comment #4 from Laura Escamilla <[email protected]> ---
(In reply to Owen Leonard from comment #2)
> This is nit-picky but I don't think wrapping the contents of that table
> header with .sr-only is correct. .sr-only is for things which are hidden
> from sighted users but readable by screen readers. If we want something
> hidden from everyone I think we have as-is.
> 
> In order to pass the accessibility test maybe we put text in that header
> cell without any class? I was thinking "Expand," which might help sighted
> users. In my test with macOS's VoiceOver the cell isn't read at all.

Thanks for the feedback that makes sense. I’ve updated the patch to use visible
text in the header instead of .sr-only, so it now reads “Expand.” This avoids
the empty <th> while also providing a clearer label for all users.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to