Multiple generic ‘Select this item’ check boxes are present on the search results pages that some users may not be able to differentiate between when navigating out of context. 1/ Search for an item 2/ Note that the results page has multiple checkboxes with matching aria tags. Current code: <td class="select selectcol"> <input name="biblionumber" class="cb" id="bib28526" aria-label="Select this item" type="checkbox" value="28526"> <label for="bib28526"></label> </td> Solution: Ensure that all form elements are uniquely descriptive of their purpose. For example: <td class="select selectcol"> <input name="biblionumber" class="cb" id="bib28526" aria-label="Select ‘Taxing time for housing’" type="checkbox" value="28526"> <label for="bib28526"></label> </td>
Hi Martin, I think I'm missing something, but I can't see aria-label for the checkboxes in the Staff client or the OPAC, not in the code on the webpage nor in the source code. Could you please let me know how I can see what you're seeing for "Current code"?
Interesting.. it looks like standard Koha is missing the aria labels entirely. Patch inbound
Created attachment 106404 [details] [review] Bug 25244: Add specific aria-label to search results This patch adds an arai-lebel to the select checkboxes on the search results page of the OPAC. Test plan 1/ Perform a search that will yield some results 2/ Inspect the 'checkbox' on the left side of the results table 3/ Note that there is no arai-label 4/ Apply patch 5/ Refresh the search results page 6/ Inspect the 'checkbox' on the left side of the results table 7/ Note that the aria-label now appears and contains relavant information 8/ Signoff
Created attachment 106406 [details] [review] Bug 25244: Add specific aria-label to search results This patch adds an arai-lebel to the select checkboxes on the search results page of the OPAC. Test plan 1/ Perform a search that will yield some results 2/ Inspect the 'checkbox' on the left side of the results table 3/ Note that there is no arai-label 4/ Apply patch 5/ Refresh the search results page 6/ Inspect the 'checkbox' on the left side of the results table 7/ Note that the aria-label now appears and contains relavant information 8/ Signoff Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Shouldn't it be "Select bibliographic record" ? I think the values of aria-label attributes are not automatically picked up by the translator, so it is not translatable. Also, QA script complains about missing TT filters
(In reply to Julian Maurice from comment #5) > Shouldn't it be "Select bibliographic record" ? Hmm, perhaps "Select search result: ".. your right it's a bib record and not an item, but to an OPAC user I'm not sure they'd want to draw the difference.. it's used for selecting a result to place on hold etc. > > I think the values of aria-label attributes are not automatically picked up > by the translator, so it is not translatable. Good question.. I didn't think about that.. is that not a wider issue though.. we need aria-labels for accessibility.. we really ought to pick them for translations. > > Also, QA script complains about missing TT filters Oops.. I'll fix that.
Created attachment 106499 [details] [review] Bug 25244: Add specific aria-label to search results This patch adds an arai-lebel to the select checkboxes on the search results page of the OPAC. Test plan 1/ Perform a search that will yield some results 2/ Inspect the 'checkbox' on the left side of the results table 3/ Note that there is no arai-label 4/ Apply patch 5/ Refresh the search results page 6/ Inspect the 'checkbox' on the left side of the results table 7/ Note that the aria-label now appears and contains relavant information 8/ Signoff Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 106500 [details] [review] Bug 25244: (follow-up) QA corrections
Added bug 25922 to track the translation tools enhancement to allow the pickup of aria-labels for translation.
Created attachment 106511 [details] [review] Bug 25244: Add specific aria-label to search results This patch adds an arai-lebel to the select checkboxes on the search results page of the OPAC. Test plan 1/ Perform a search that will yield some results 2/ Inspect the 'checkbox' on the left side of the results table 3/ Note that there is no arai-label 4/ Apply patch 5/ Refresh the search results page 6/ Inspect the 'checkbox' on the left side of the results table 7/ Note that the aria-label now appears and contains relavant information 8/ Signoff Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Created attachment 106512 [details] [review] Bug 25244: (follow-up) QA corrections Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Can we switch item for something else please? In German item (for holdings) and item (like a list item) are not the same word and there is not enough context to make sense for translation here.
(In reply to Katrin Fischer from comment #12) > Can we switch item for something else please? In German item (for holdings) > and item (like a list item) are not the same word and there is not enough > context to make sense for translation here. I realized now, that the follow-up took care of this.
Created attachment 107109 [details] [review] Bug 25244: Add specific aria-label to search results This patch adds an arai-lebel to the select checkboxes on the search results page of the OPAC. Test plan 1/ Perform a search that will yield some results 2/ Inspect the 'checkbox' on the left side of the results table 3/ Note that there is no arai-label 4/ Apply patch 5/ Refresh the search results page 6/ Inspect the 'checkbox' on the left side of the results table 7/ Note that the aria-label now appears and contains relavant information 8/ Signoff Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Julian Maurice <julian.maurice@biblibre.com> Bug 25244: (follow-up) QA corrections Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Patches squashed.
Pushed to master for 20.11, thanks to everybody involved!
backported to 20.05.x for 20.05.03
backported to 19.11.x for 19.11.09
Backported to 19.05.x branch for 19.05.15