https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 --- Comment #2 from Owen Leonard <oleonard@myacpl.org> --- 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. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 --- Comment #3 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- Created attachment 196497 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196497&action=edit Bug 41934: Add label to responsive control column header in OPAC course reserves The responsive control column in the OPAC course reserves table contained an empty <th>, which results in an empty header and accessibility warnings from automated tools. This patch adds visible text ("Expand") to the header cell to provide a meaningful label for all users and resolve the issue. To test: 1. Enable course reserves and add at least one course with items. 2. Visit the OPAC course reserves page. 3. Inspect the table header row. 4. Confirm the responsive control column now contains "Expand". 5. Verify no empty header warnings are reported by accessibility tools. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 Brendan Lawlor <blawlor@clamsnet.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply CC| |blawlor@clamsnet.org --- Comment #5 from Brendan Lawlor <blawlor@clamsnet.org> --- I tried to test this out but got a sha1 error, so I could not apply the patch. A couple other things I think this would need for better accessibility is the +/- element should probably be a <button> element and have aria-expanded attribute set to false initially and updated to true when the notes expand. It should have an aria-controls attribute that refers to the the id of the notes element that is expanded. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196497|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 --- Comment #7 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- (In reply to Brendan Lawlor from comment #5)

I tried to test this out but got a sha1 error, so I could not apply the patch.

A couple other things I think this would need for better accessibility is the +/- element should probably be a <button> element and have aria-expanded attribute set to false initially and updated to true when the notes expand. It should have an aria-controls attribute that refers to the the id of the notes element that is expanded.

Thanks, I agree those would be worthwhile accessibility improvements. For this patch, I was aiming to address the empty table header issue specifically. Since the expand/collapse control is generated through DataTables responsive behavior, converting it to a button with aria-expanded and aria-controls would be a broader change than this bug currently covers. I think that would be better tracked as a follow-up bug so this patch can remain focused on the empty header fix. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196687|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This fixes an accessibility release notes| |issue with the course | |reserves table in the OPAC | |(OPAC > Course reserves): | |"Table header text should | |not be empty". | | | |Previously when responsive | |table controls were shown | |for the list of course | |reserves (a green "+" | |button is shown when the | |browser window is narrower | |and all the columns can't | |be displayed), there was no | |title for the column header | |with the controls. | | | |Now, the column header has | |the text "Expand" when the | |responsive controls are | |shown. CC| |david@davidnind.com --- Comment #9 from David Nind <david@davidnind.com> --- Testing notes (using KTD): 1. Before the patch, go to the OPAC > Course reserves (which lists available course reserves) 2. Narrow the width of your browser so that the responsive controls for the table appear (a green button with a +). 3. Note that the column the green + button appears in has no column title. 4. Apply the patch. 5. Refresh the page and repeat step 2. 6. Note that there is now a column title "Expand". Using the axe DevTools extension: 1. Before the patch, with responsive tables shown, there is one minor issue: "Table header text should not be empty" 2. After the patch, with responsive table controls shown: there is no longer an issue for this. 3. Note that there are other accessibility issues on the page, but these are outside the scope of this bug. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 Brendan Lawlor <blawlor@clamsnet.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196956|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to main Version(s)| |26.05.00 released in| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 Raina <raina@flo.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |raina@flo.org -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41934 --- Comment #12 from Jacob O'Mara <jacob.omara@openfifth.co.uk> --- Thanks all, pushed to 25.11.x -- You are receiving this mail because: You are watching all bug changes.