Summary: | Improve layout of course reserve details | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | Pushed to main --- | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | david, m.de.rooy |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Small patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: |
This enhancement improves the layout of course reserves pages.
For the list of courses, it changes the "Reserves" heading to "Courses".
For the course details page it:
- Replaces the "Reserves" heading with the course name,
matching the breadcrumbs and page title.
- Adds separator lines between fields.
- Adds colons after labels.
- Fixes some alignment issues.
- Only displays fields with values.
- Now shows "0" for the "Number of students" if no value is added.
|
|
Version(s) released in: |
25.05.00
|
Circulation function: | |
Attachments: |
Bug 38351: Improve layout of course reserve details
Screenshot - Course details page(showing formatting issues) Bug 38351: (follow-up) Tweaks to list and form alignment Bug 38351: Improve layout of course reserve details Bug 38351: (follow-up) Tweaks to list and form alignment Bug 38351: (follow-up) Tweaks to list and form alignment Bug 38351: Improve layout of course reserve details Bug 38351: (follow-up) Tweaks to list and form alignment Bug 38351: Improve layout of course reserve details Bug 38351: (follow-up) Tweaks to list and form alignment |
Description
Owen Leonard
2024-11-04 17:40:56 UTC
Created attachment 173942 [details] [review] Bug 38351: Improve layout of course reserve details This patch makes some simple markup and CSS changes to the course details page in order to improve the layout of the section: - Add colons after labels. - Replace fieldset.rows with div.rows (there's no form). - Make list of instructors line up with other details. - Do not display fields which are empty. ALSO: Replaced the "Reserves" heading with course name, matching breadcrumbs and page title. To test, apply the patch and go to Course reserves. - If necessary, create a course. - View the details of the course. - Confirm that the list of details looks correct. - Test with all the optional fields empty: Section, term, instructor, staff note, public note. There should not be empty lines for these fields in the course details. - "Number of students" should show "0" if you left the field empty. - Add multiple instructors to confirm that they are displayed in alignment with other course details. - The page heading should match the page breadcrumb. Sponsored-by: Athens County Public Libraries Created attachment 176109 [details]
Screenshot - Course details page(showing formatting issues)
Hi Owen.
This is a great improvement!
However, some things I noticed:
1. There is a row border just above/through the instructors row. (See screenshot.)
2. The staff note and public note are misaligned. (See screenshot.)
3. When editing the course, the instructor search label and search box are indented/misaligned, unless that is the way it is supposed to be.
David
Created attachment 176389 [details] [review] Bug 38351: (follow-up) Tweaks to list and form alignment Created attachment 176394 [details] [review] Bug 38351: Improve layout of course reserve details This patch makes some simple markup and CSS changes to the course details page in order to improve the layout of the section: - Add colons after labels. - Replace fieldset.rows with div.rows (there's no form). - Make list of instructors line up with other details. - Do not display fields which are empty. ALSO: Replaced the "Reserves" heading with course name, matching breadcrumbs and page title. To test, apply the patch and go to Course reserves. - If necessary, create a course. - View the details of the course. - Confirm that the list of details looks correct. - Test with all the optional fields empty: Section, term, instructor, staff note, public note. There should not be empty lines for these fields in the course details. - "Number of students" should show "0" if you left the field empty. - Add multiple instructors to confirm that they are displayed in alignment with other course details. - The page heading should match the page breadcrumb. Sponsored-by: Athens County Public Libraries Signed-off-by: David Nind <david@davidnind.com> Created attachment 176395 [details] [review] Bug 38351: (follow-up) Tweaks to list and form alignment Signed-off-by: David Nind <david@davidnind.com> Thanks Owen! Created attachment 176396 [details] [review] Bug 38351: (follow-up) Tweaks to list and form alignment Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: William Lavoie <william.lavoie@inLibro.com> Created attachment 177917 [details] [review] Bug 38351: Improve layout of course reserve details This patch makes some simple markup and CSS changes to the course details page in order to improve the layout of the section: - Add colons after labels. - Replace fieldset.rows with div.rows (there's no form). - Make list of instructors line up with other details. - Do not display fields which are empty. ALSO: Replaced the "Reserves" heading with course name, matching breadcrumbs and page title. To test, apply the patch and go to Course reserves. - If necessary, create a course. - View the details of the course. - Confirm that the list of details looks correct. - Test with all the optional fields empty: Section, term, instructor, staff note, public note. There should not be empty lines for these fields in the course details. - "Number of students" should show "0" if you left the field empty. - Add multiple instructors to confirm that they are displayed in alignment with other course details. - The page heading should match the page breadcrumb. Sponsored-by: Athens County Public Libraries Signed-off-by: David Nind <david@davidnind.com> Created attachment 177918 [details] [review] Bug 38351: (follow-up) Tweaks to list and form alignment Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: William Lavoie <william.lavoie@inLibro.com> Created attachment 179053 [details] [review] Bug 38351: Improve layout of course reserve details This patch makes some simple markup and CSS changes to the course details page in order to improve the layout of the section: - Add colons after labels. - Replace fieldset.rows with div.rows (there's no form). - Make list of instructors line up with other details. - Do not display fields which are empty. ALSO: Replaced the "Reserves" heading with course name, matching breadcrumbs and page title. To test, apply the patch and go to Course reserves. - If necessary, create a course. - View the details of the course. - Confirm that the list of details looks correct. - Test with all the optional fields empty: Section, term, instructor, staff note, public note. There should not be empty lines for these fields in the course details. - "Number of students" should show "0" if you left the field empty. - Add multiple instructors to confirm that they are displayed in alignment with other course details. - The page heading should match the page breadcrumb. Sponsored-by: Athens County Public Libraries Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 179054 [details] [review] Bug 38351: (follow-up) Tweaks to list and form alignment Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: William Lavoie <william.lavoie@inLibro.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> The release notes seem to have some non-mark-down mark-up, can you please check and revise? Pushed for 25.05! Well done everyone, thank you! (In reply to Katrin Fischer from comment #12) > The release notes seem to have some non-mark-down mark-up, can you please > check and revise? I didn't release we could use markdown "markup". Is there any specific guidance about this (markdown flavour, what works and what doesn't)? I've simplified the list of improvements a bit, if this helps. |