Summary: | Add column configuration to course reserves | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | caroline.cyr-la-rose, jonathan.druart, martin.renvoize, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 19282 | ||
Attachments: |
Bug 21409: Add column configuration to course reserves
Bug 21409: Add column configuration to course reserves Bug 21409: Add column configuration to course reserves |
Description
Owen Leonard
2018-09-25 18:39:39 UTC
Created attachment 79383 [details] [review] Bug 21409: Add column configuration to course reserves This patch adds columns configuration and print/export options to two tables: The table listing courses and the table listing titles reserved for a course. To test you should have more than one course and multiple title reserved for at least one course. - Apply the patch and go to Course reserves - In the table of courses, confirm that all DataTables controls work correctly: paging, filter, column visibility, export, etc. - View a course with multiple titles reserved. Test all DataTables controls on this table too. - Go to Administration -> Columns settings - Change the settings for the courses and reserves table and confirm that these changes are applied correctly. Created attachment 79737 [details] [review] Bug 21409: Add column configuration to course reserves This patch adds columns configuration and print/export options to two tables: The table listing courses and the table listing titles reserved for a course. To test you should have more than one course and multiple title reserved for at least one course. - Apply the patch and go to Course reserves - In the table of courses, confirm that all DataTables controls work correctly: paging, filter, column visibility, export, etc. - View a course with multiple titles reserved. Test all DataTables controls on this table too. - Go to Administration -> Columns settings - Change the settings for the courses and reserves table and confirm that these changes are applied correctly. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> I have a small concern: we remove the table's caption. We should use it instead of moving it to a h1 (and it's too late for a h1 here, it must be h2 or h3). I tried to move the caption on top of the DT paging/filtering/buttons block, but failed. It seems that it's not feasible. Maybe we should restyle the caption to reduce the space between the "pager" and the table (I guess it's why you move it out). Caption - https://snag.gy/mikSJI.jpg h1- https://snag.gy/sLIK3u.jpg Caption adjusted (text-align: center; font-size: inherit; margin: inherit;) - https://snag.gy/GYahyl.jpg (In reply to Jonathan Druart from comment #3) > I have a small concern: we remove the table's caption. We have neither a coding guideline regarding captions nor one defining how to use heading tags, so the templates are pretty random with regard to these elements. I think most if not all tables with DataTables toolbars use a heading instead of a caption. Created attachment 79820 [details] [review] Bug 21409: Add column configuration to course reserves This patch adds columns configuration and print/export options to two tables: The table listing courses and the table listing titles reserved for a course. To test you should have more than one course and multiple title reserved for at least one course. - Apply the patch and go to Course reserves - In the table of courses, confirm that all DataTables controls work correctly: paging, filter, column visibility, export, etc. - View a course with multiple titles reserved. Test all DataTables controls on this table too. - Go to Administration -> Columns settings - Change the settings for the courses and reserves table and confirm that these changes are applied correctly. Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> (In reply to Owen Leonard from comment #5) > (In reply to Jonathan Druart from comment #3) > > I have a small concern: we remove the table's caption. > > We have neither a coding guideline regarding captions nor one defining how > to use heading tags, so the templates are pretty random with regard to these > elements. I think most if not all tables with DataTables toolbars use a > heading instead of a caption. https://www.w3.org/TR/WCAG20-TECHS/H39.html """ The objective of this technique is to programmatically associate captions for data tables where captions are provided in the presentation. The caption for a table is a table identifier and acts like a title or heading for the table. The caption element is the appropriate markup for such text and it ensures that the table identifier remains associated with the table, including visually (by default). In addition, using the caption element allows screen reading software to navigate directly to the caption for a table if one is present. """ We should use captions :) Awesome work all! Pushed to master for 18.11 Enhancement will not be backported to 18.05.x series. |