Bug 27741

Summary: Accessibility: Staff Client - Headings should have correct tags and hierarchy
Product: Koha Reporter: wainuiwitikapark
Component: AccessibilityAssignee: wainuiwitikapark
Status: BLOCKED --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: claire.hernandez, gmcharlt, jonathan.druart, lucas, martin.renvoize, matt.blenkinsop, oleonard
Version: unspecifiedKeywords: accessibility
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Large patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 27631    
Bug Blocks: 27877, 28243, 28532    
Attachments: Bug 27741: admin folder
Bug 27741: admin folder
Bug 27741: admin folder
Bug 27741: admin folder
Bug 27741: authorities folder
Bug 27741: basket folder
Bug 27741: admin folder
Bug 27741: authorities folder
Bug 27741: basket folder
Bug 27741: catalogue folder
Bug 27741: cataloguing folder
Bug 27741: clubs folder
Bug 27741: common folder
Bug 27741: course_reserves folder
Bug 27741: virtualshelves folder
Bug 27741: patron_lists folder
Bug 27741: pos folder
Bug 27741: suggestion folder
Bug 27741: tags folder
Bug 27741: patroncards folder
Bug 27741: labels folder
Bug 27741: members folder
Bug 27741: serials folder
Bug 27741: reports folder
Bug 27741: circ folder
Bug 27741: tools folder
Bug 27741: acqui folder
Bug 27741: onboarding folder
Bug 27741: reserve folder
Bug 27741: about.tt
Bug 27741: main page
Bug 27741: tools folder
Bug 27741: installer folder
Bug 27741: ill folder
Bug 27741: circ folder
Bug 27741: admin folder
Bug 27741: authorities folder
Bug 27741: basket folder
Bug 27741: catalogue folder
Bug 27741: cataloguing folder
Bug 27741: clubs folder
Bug 27741: common folder
Bug 27741: virtualshelves folder
Bug 27741: patron_lists folder
Bug 27741: pos folder
Bug 27741: suggestion folder
Bug 27741: tags folder
Bug 27741: patroncards folder
Bug 27741: labels folder
Bug 27741: serials folder
Bug 27741: onboarding folder
Bug 27741: about.tt
Bug 27741: main page
Bug 27741: installer folder
Bug 27741: ill folder
Bug 27741: tools folder
Bug 27741: circ folder
Bug 27741: reserve folder
Bug 27741: reports folder
Bug 27741: acqui folder
Bug 27741: members folder
Bug 27741: course_reserves folder

Description wainuiwitikapark 2021-02-21 21:07:15 UTC
Each page of the Staff Client should contain correct headings with correct heading tags and hierarchy. Please see https://www.w3.org/WAI/tutorials/page-structure/headings/ for more information.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27631 dealt with changing the logo on each page to be a <div> instead of an <h1> and then ensuring each page has one <h1> that tells users what the page is about.

This bug will deal with ensuring the rest of the headings in each page follow the correct heading hierarchy, and headings don't skip a level.
Comment 1 wainuiwitikapark 2021-04-27 07:07:13 UTC Comment hidden (obsolete)
Comment 2 wainuiwitikapark 2021-04-27 07:15:33 UTC Comment hidden (obsolete)
Comment 3 wainuiwitikapark 2021-04-27 07:17:15 UTC
I changed the admin menu headings to <h2>. These can be amended in the scss files if required.
Comment 4 wainuiwitikapark 2021-04-27 07:22:38 UTC Comment hidden (obsolete)
Comment 5 wainuiwitikapark 2021-04-27 07:46:16 UTC Comment hidden (obsolete)
Comment 6 wainuiwitikapark 2021-04-27 07:48:02 UTC
I changed the includes/admin-menu.inc headings and includes/acquisitions-menu.inc headings to <h2>. These are large but can be amended in the scss files if required.
Comment 7 wainuiwitikapark 2021-04-27 08:01:41 UTC Comment hidden (obsolete)
Comment 8 wainuiwitikapark 2021-04-27 08:06:03 UTC Comment hidden (obsolete)
Comment 9 wainuiwitikapark 2021-05-25 01:22:29 UTC
Created attachment 121344 [details] [review]
Bug 27741: admin folder

Bug 27631 corrected the <h1> tags in each of the pages in the staf client. This bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 10 wainuiwitikapark 2021-05-25 01:43:13 UTC
Created attachment 121346 [details] [review]
Bug 27741: authorities folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff client. This bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 11 wainuiwitikapark 2021-05-25 01:46:57 UTC
Created attachment 121347 [details] [review]
Bug 27741: basket folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 12 wainuiwitikapark 2021-05-25 02:58:34 UTC
Created attachment 121348 [details] [review]
Bug 27741: catalogue folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 13 wainuiwitikapark 2021-05-25 03:57:49 UTC
Created attachment 121349 [details] [review]
Bug 27741: cataloguing folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 14 wainuiwitikapark 2021-05-25 07:07:34 UTC
Created attachment 121350 [details] [review]
Bug 27741: clubs folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 15 wainuiwitikapark 2021-05-25 07:13:38 UTC
Created attachment 121351 [details] [review]
Bug 27741: common folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 16 wainuiwitikapark 2021-05-25 08:12:10 UTC
Created attachment 121357 [details] [review]
Bug 27741: course_reserves folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 17 wainuiwitikapark 2021-05-25 08:32:08 UTC
Created attachment 121358 [details] [review]
Bug 27741: virtualshelves folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 18 wainuiwitikapark 2021-05-25 08:50:03 UTC
Created attachment 121365 [details] [review]
Bug 27741: patron_lists folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 19 wainuiwitikapark 2021-05-25 23:46:20 UTC
Created attachment 121400 [details] [review]
Bug 27741: pos folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 20 wainuiwitikapark 2021-05-26 02:25:38 UTC
Created attachment 121402 [details] [review]
Bug 27741: suggestion folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 21 wainuiwitikapark 2021-05-26 02:35:05 UTC
Created attachment 121403 [details] [review]
Bug 27741: tags folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 22 wainuiwitikapark 2021-05-26 02:58:34 UTC
Created attachment 121404 [details] [review]
Bug 27741: patroncards folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 23 wainuiwitikapark 2021-05-26 03:04:52 UTC
So yeah, with all the menus and things on the sides, I have made them <h2>. These can be amended in the css to make them appear smaller as they were before.
Comment 24 wainuiwitikapark 2021-05-30 04:05:59 UTC
Created attachment 121509 [details] [review]
Bug 27741: labels folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 25 wainuiwitikapark 2021-05-30 05:35:41 UTC
Created attachment 121510 [details] [review]
Bug 27741: members folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 26 wainuiwitikapark 2021-06-01 01:23:19 UTC
Created attachment 121521 [details] [review]
Bug 27741: serials folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 27 wainuiwitikapark 2021-06-02 03:36:49 UTC
Created attachment 121534 [details] [review]
Bug 27741: reports folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 28 wainuiwitikapark 2021-06-03 03:08:45 UTC Comment hidden (obsolete)
Comment 29 wainuiwitikapark 2021-06-04 01:48:40 UTC Comment hidden (obsolete)
Comment 30 wainuiwitikapark 2021-06-06 04:52:29 UTC
Created attachment 121642 [details] [review]
Bug 27741: acqui folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 31 wainuiwitikapark 2021-06-06 05:08:37 UTC
Created attachment 121643 [details] [review]
Bug 27741: onboarding folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 32 wainuiwitikapark 2021-06-06 05:32:51 UTC
Created attachment 121644 [details] [review]
Bug 27741: reserve folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 33 wainuiwitikapark 2021-06-06 10:50:49 UTC
Created attachment 121645 [details] [review]
Bug 27741: about.tt

Bug 27631 corrected the <h1> tags in each of the pages in the staff client. This bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 34 wainuiwitikapark 2021-06-06 10:57:41 UTC
Created attachment 121646 [details] [review]
Bug 27741: main page

Bug 27631 corrected the <h1> tags in each of the pages in the staff client. This bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to the main page of the staff client and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 35 wainuiwitikapark 2021-06-08 04:23:45 UTC
Created attachment 121699 [details] [review]
Bug 27741: tools folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 36 wainuiwitikapark 2021-06-08 05:27:18 UTC
Created attachment 121700 [details] [review]
Bug 27741: installer folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff client. This bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 37 wainuiwitikapark 2021-06-08 05:47:30 UTC
Created attachment 121701 [details] [review]
Bug 27741: ill folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff client. This bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 38 wainuiwitikapark 2021-06-16 01:58:53 UTC
Created attachment 122018 [details] [review]
Bug 27741: circ folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT
Comment 39 Owen Leonard 2021-09-30 16:51:41 UTC
I'm not sure if it was my rebase of Bug 27631 or something else, but this doesn't apply: 

Applying: Bug 27741: admin folder
error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/admin/patron-attr-types.tt).
error: could not build fake ancestor
Patch failed at 0001 Bug 27741: admin folder
Comment 40 Owen Leonard 2021-11-05 14:57:39 UTC
Created attachment 127369 [details] [review]
Bug 27741: admin folder

Bug 27631 corrected the <h1> tags in each of the pages in the staf
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
   correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 41 Owen Leonard 2021-11-05 14:57:43 UTC
Created attachment 127370 [details] [review]
Bug 27741: authorities folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff client. This
bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 42 Owen Leonard 2021-11-05 14:57:47 UTC
Created attachment 127371 [details] [review]
Bug 27741: basket folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 43 Owen Leonard 2021-11-05 14:57:50 UTC
Created attachment 127372 [details] [review]
Bug 27741: catalogue folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 44 Owen Leonard 2021-11-05 14:57:54 UTC
Created attachment 127373 [details] [review]
Bug 27741: cataloguing folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 45 Owen Leonard 2021-11-05 14:57:58 UTC
Created attachment 127374 [details] [review]
Bug 27741: clubs folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 46 Owen Leonard 2021-11-05 14:58:02 UTC
Created attachment 127375 [details] [review]
Bug 27741: common folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 47 Owen Leonard 2021-11-05 14:58:06 UTC
Created attachment 127376 [details] [review]
Bug 27741: virtualshelves folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 48 Owen Leonard 2021-11-05 14:58:10 UTC
Created attachment 127377 [details] [review]
Bug 27741: patron_lists folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 49 Owen Leonard 2021-11-05 14:58:14 UTC
Created attachment 127378 [details] [review]
Bug 27741: pos folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 50 Owen Leonard 2021-11-05 14:58:18 UTC
Created attachment 127379 [details] [review]
Bug 27741: suggestion folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 51 Owen Leonard 2021-11-05 14:58:23 UTC
Created attachment 127380 [details] [review]
Bug 27741: tags folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 52 Owen Leonard 2021-11-05 14:58:27 UTC
Created attachment 127381 [details] [review]
Bug 27741: patroncards folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 53 Owen Leonard 2021-11-05 14:58:31 UTC
Created attachment 127382 [details] [review]
Bug 27741: labels folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 54 Owen Leonard 2021-11-05 14:58:35 UTC
Created attachment 127383 [details] [review]
Bug 27741: serials folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 55 Owen Leonard 2021-11-05 14:58:39 UTC
Created attachment 127384 [details] [review]
Bug 27741: onboarding folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 56 Owen Leonard 2021-11-05 14:58:44 UTC
Created attachment 127385 [details] [review]
Bug 27741: about.tt

Bug 27631 corrected the <h1> tags in each of the pages in the staff client. This
bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 57 Owen Leonard 2021-11-05 14:58:48 UTC
Created attachment 127386 [details] [review]
Bug 27741: main page

Bug 27631 corrected the <h1> tags in each of the pages in the staff client. This
bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to the main page of the staff client and check the headings under <h1> are
   in the correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 58 Owen Leonard 2021-11-05 14:58:52 UTC
Created attachment 127387 [details] [review]
Bug 27741: installer folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff client. This
bug deals with correcting the hierarchy of each of the other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the correct
   order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 59 Owen Leonard 2021-11-05 14:58:56 UTC
Created attachment 127388 [details] [review]
Bug 27741: ill folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in
   the correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 60 Owen Leonard 2021-11-05 14:59:00 UTC
Created attachment 127389 [details] [review]
Bug 27741: tools folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
   correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 61 Owen Leonard 2021-11-05 14:59:04 UTC
Created attachment 127390 [details] [review]
Bug 27741: circ folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
   correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 62 Owen Leonard 2021-11-05 14:59:07 UTC
Created attachment 127391 [details] [review]
Bug 27741: reserve folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
   correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 63 Owen Leonard 2021-11-05 14:59:12 UTC
Created attachment 127392 [details] [review]
Bug 27741: reports folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
   correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 64 Owen Leonard 2021-11-05 14:59:19 UTC
Created attachment 127393 [details] [review]
Bug 27741: acqui folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
   correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 65 Owen Leonard 2021-11-05 14:59:23 UTC
Created attachment 127394 [details] [review]
Bug 27741: members folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
   correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 66 Owen Leonard 2021-11-05 14:59:29 UTC
Created attachment 127395 [details] [review]
Bug 27741: course_reserves folder

Bug 27631 corrected the <h1> tags in each of the pages in the staff
client. This bug deals with correcting the hierarchy of each of the
other headings.

To test:
1) Apply patch and dependencies
2) Go to each of the pages and check the headings under <h1> are in the
   correct order and hierarchy

Sponsored-by: Catalyst IT

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 67 Jonathan Druart 2021-12-01 15:40:25 UTC
Comment on attachment 127372 [details] [review]
Bug 27741: catalogue folder

Review of attachment 127372 [details] [review]:
-----------------------------------------------------------------

<h2>[% title | html %]</h2>
-    [% IF ( subtitle ) %]<h4>[% FOREACH subtitl IN subtitle %] [% subtitl | html %][% END %]</h4>[% END %]
-    [% IF ( author ) %]<h4>by [% author | html %]</h4>[% END %]
+    <h2>
+    [% IF ( subtitle ) %][% FOREACH subtitl IN subtitle %] [% subtitl | html %][% END %][% END %]
+    [% IF ( author ) %]by [% author | html %][% END %]
+    </h2>

Is this correct? Shouldn't be h3?
Comment 68 Jonathan Druart 2021-12-01 15:42:22 UTC
Impressive work Wainui!
How did you process? Has all been done manually or you managed to catch them using a specific tool?
We will certainly need something automated once pushed to catch regressions.
Comment 69 Jonathan Druart 2022-02-02 09:09:37 UTC
Wainui, ping?
Comment 70 Jonathan Druart 2022-02-18 14:36:59 UTC
Blocked by bug 27631.