| Summary: | Empty table header in course reserves table causes accessibility error | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Laura Escamilla <Laura.escamilla> |
| Component: | Accessibility | Assignee: | Laura Escamilla <Laura.escamilla> |
| Status: | Needs Signoff --- | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | martin.renvoize, matt.blenkinsop, oleonard |
| Version: | Main | Keywords: | accessibility |
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: | Bug 41934: Fix empty table header in course reserves table | ||
|
Description
Laura Escamilla
2026-02-25 15:51:21 UTC
Created attachment 193914 [details] [review] Bug 41934: Fix empty table header in course reserves table To test: 1. Apply the patch. 2. Navigate to the OPAC course reserves page: http://localhost:8080/cgi-bin/koha/opac-course-reserves.pl 3. Inspect the course reserves table header using browser developer tools. 4. Confirm the responsive control column header contains visually hidden text describing the column (e.g. "Details"). 5. Confirm the table appearance and responsive expand/collapse behavior remain unchanged. 6. Sign off. 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. |