Library Catalogue - OPAC The ‘Confirm reservations for: Person X (111000111)’ heading is not logical because it increases by more than one heading level. This means that screen reader users may not be able to determine the structure of content on the page. 1/ Search for an item and attempt to reserve it Current Code: <h3>Confirm reservations for: Person X (111000111)</h3>
Created attachment 105741 [details] [review] Bug 25239: Fix semantic heading for opac-reserve.tt Prior to this patch, the opac-reserve contained semantically incorrect headings. Test plan 1/ Perform a search which yields some results in the OPAC 2/ Attempt to place a hold on an item. 3/ On the resulting screen, note that the headings are: H1 (Koha page title) > H3 (Confirm holds for) > H5 (Hold levels) 4/ Apply this patch and reload the page 5/ Re-inspect the elements in question note the heading semantics are now: H1 (Title of page) > H2 (Confirm holds for) > H3 (Hold levels) 6/ Note the appearance of the box is still reasonable 7/ Signoff
Created attachment 105758 [details] [review] Bug 25239: Fix semantic heading for opac-reserve.tt Prior to this patch, the opac-reserve contained semantically incorrect headings. Test plan 1/ Perform a search which yields some results in the OPAC 2/ Attempt to place a hold on an item. 3/ On the resulting screen, note that the headings are: H1 (Koha page title) > H3 (Confirm holds for) > H5 (Hold levels) 4/ Apply this patch and reload the page 5/ Re-inspect the elements in question note the heading semantics are now: H1 (Title of page) > H2 (Confirm holds for) > H3 (Hold levels) 6/ Note the appearance of the box is still reasonable 7/ Signoff Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
Works great - just a note for future testers, to make H3 (Hold levels) appear, do the following: 1/ Perform a search which yields some results in the OPAC 2/ Place a hold on an item 3/ Go back to the same record and attempt to place a second hold 4/ On the resulting screen, note that the headings are: H1 (Koha page title) > H3 (Confirm holds for) > H5 (Hold levels) 5/ Apply this patch and reload the page 6/ Re-inspect the elements in question note the heading semantics are now: H1 (Title of page) > H2 (Confirm holds for) > H3 (Hold levels) 7/ Note the appearance of the box is still reasonable 8/ Signoff
Created attachment 106116 [details] [review] Bug 25239: Fix semantic heading for opac-reserve.tt Prior to this patch, the opac-reserve contained semantically incorrect headings. Test plan 1/ Perform a search which yields some results in the OPAC 2/ Attempt to place a hold on an item. 3/ On the resulting screen, note that the headings are: H1 (Koha page title) > H3 (Confirm holds for) > H5 (Hold levels) 4/ Apply this patch and reload the page 5/ Re-inspect the elements in question note the heading semantics are now: H1 (Title of page) > H2 (Confirm holds for) > H3 (Hold levels) 6/ Note the appearance of the box is still reasonable 7/ Signoff Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 20.11, thanks to everybody involved!
backported to 20.05.x for 20.05.02
There are no <h5> headings on this page in 19.11.x. I'd like to backport this for accessibility, but need someone to write a patch that fixes the <h3>Confirm holds for:</h3> for 19.11.x only.
decided to rebase myself :) backported to 19.11.x for 19.11.08
Backported to 19.05.x branch for 19.05.14