Bug 27631 created <h1> headings on each page of the Staff Client, then Bug 27741 fixed the hierarchy of the other headings in each page to ensure there was no skipping of heading levels, and that items that appeared as headings of some sort, had an <h> tag. There are now a few headings that are redundant so this bug will deal with issues like this. Many of the alerts and errors are also tagged as headings, which for many cases is not correct. Therefore, this bug will also deal with these issues
I don't think this bug needs to address error alerts anymore
Files I specifically noticed may have redundancy: admin/classsources.tt circ/set-library.tt circ/circulation-home.tt pos/pay.tt
The only file/page I noticed unnecessary was circ/set-library.tt Adding patch to amend this now... Please let me know if you notice anymore redundant headings that have appeared because of my accessibility heading fixes.
Created attachment 121725 [details] [review] Bug 27877: circ/set-library.tt Got rid of heading redundancy To test: 1) Apply patch and dependencies 2) Go to Staff Client 3) Go to circ/set-library 4) Check that there are no visible unnecessary redundant/repeated headings 5) Check that there is a legend that has the class "sr-only" which eliminates visible redundancy but leaves descriptions for those who use screen readers Sponsored-by: Catalyst IT
Could you just explain the problem before the patch please so we can see the difference between the code before and after?