The responsive control column in the OPAC course reserves table contains an empty <th> element. This results in an empty table header and an empty aria-label, which is flagged as an accessibility error by automated testing tools.
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.