Summary: | The basket summary page template needs a cleanup | ||
---|---|---|---|
Product: | Koha | Reporter: | Martin Renvoize (ashimema) <martin.renvoize> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Katrin Fischer <katrin.fischer> |
Severity: | minor | ||
Priority: | P5 - low | CC: | gmcharlt, lucas |
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: |
23.05.00,22.11.03
|
|
Circulation function: | |||
Bug Depends on: | 31837 | ||
Bug Blocks: | 32733 | ||
Attachments: |
Bug 31932: The basket summary page template needs a cleanup
Bug 31932: (follow-up) Markup comments Bug 31932: The basket summary page template needs a cleanup Bug 31932: (follow-up) Markup comments Bug 31932: The basket summary page template needs a cleanup Bug 31932: (follow-up) Markup comments |
Description
Martin Renvoize (ashimema)
2022-10-24 08:32:24 UTC
Created attachment 145664 [details] [review] Bug 31932: The basket summary page template needs a cleanup This patch reindents the Acquisitions basket template so that it has consistent indentation. These changes should have no visible effect on the page. To test, apply the patch and go to Acquisitions. Locate a vendor and add a basket and some orders if necessary. Test as much of the page's functionality as possible, e.g. toolbar controls, in-table controls, managed-by and managing library controls, etc. Try adding an internal note, adding a vendor note, and deleting the basket. Everything should look correct and work correctly. Viewing the diff while ignoring whitespace changes should show only places where a line break was added. Created attachment 145665 [details] [review] Bug 31932: (follow-up) Markup comments This patch adds comments to the template to highlight the markup structure. This patch should have no effect on the page's appearance or functionality. Created attachment 145686 [details] [review] Bug 31932: The basket summary page template needs a cleanup This patch reindents the Acquisitions basket template so that it has consistent indentation. These changes should have no visible effect on the page. To test, apply the patch and go to Acquisitions. Locate a vendor and add a basket and some orders if necessary. Test as much of the page's functionality as possible, e.g. toolbar controls, in-table controls, managed-by and managing library controls, etc. Try adding an internal note, adding a vendor note, and deleting the basket. Everything should look correct and work correctly. Viewing the diff while ignoring whitespace changes should show only places where a line break was added. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 145687 [details] [review] Bug 31932: (follow-up) Markup comments This patch adds comments to the template to highlight the markup structure. This patch should have no effect on the page's appearance or functionality. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Much clearer, thanks for this Owen.. I had meant to come back to it myself but there's never enough time is there! It's much more understandable now. Signing off Created attachment 145736 [details] [review] Bug 31932: The basket summary page template needs a cleanup This patch reindents the Acquisitions basket template so that it has consistent indentation. These changes should have no visible effect on the page. To test, apply the patch and go to Acquisitions. Locate a vendor and add a basket and some orders if necessary. Test as much of the page's functionality as possible, e.g. toolbar controls, in-table controls, managed-by and managing library controls, etc. Try adding an internal note, adding a vendor note, and deleting the basket. Everything should look correct and work correctly. Viewing the diff while ignoring whitespace changes should show only places where a line break was added. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> Created attachment 145737 [details] [review] Bug 31932: (follow-up) Markup comments This patch adds comments to the template to highlight the markup structure. This patch should have no effect on the page's appearance or functionality. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> Pushed to master for 23.05. Nice work everyone, thanks! Nice work everyone! Pushed to stable for 22.11.x Missing dependencies for 22.05.x, no backport |