I propose to reformat the tools-home template and the tools-menu include file in the staff interface so that they have consistent spacing and indentation.
Created attachment 140990 [details] [review] Bug 31625: Reindent tools home and tools sidebar This patch performs general template cleanup to the tools-home and tools-menu templates: Make indentation consistent, replace tabs with spaces, fix invalid HTML. To test, apply the patch and go to Tools. The page should look correct, with no visible changes from before the patch. Go to any tools page which shows the left-hand sidebar, e.g. Patron lists, Batch item deletion, Inventory, etc. The menu should look correct.
Created attachment 140991 [details] [review] Bug 31625: (follow-up) Markup comments This patch adds comments to the templates to highlight the markup structure. This patch should have no effect on the pages' appearance or functionality.
Created attachment 141002 [details] [review] Bug 31625: Reindent tools home and tools sidebar This patch performs general template cleanup to the tools-home and tools-menu templates: Make indentation consistent, replace tabs with spaces, fix invalid HTML. To test, apply the patch and go to Tools. The page should look correct, with no visible changes from before the patch. Go to any tools page which shows the left-hand sidebar, e.g. Patron lists, Batch item deletion, Inventory, etc. The menu should look correct. Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Created attachment 141003 [details] [review] Bug 31625: (follow-up) Markup comments This patch adds comments to the templates to highlight the markup structure. This patch should have no effect on the pages' appearance or functionality. Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Hm, I wonder if this does not clash with 31162 and if we should push one or the other first?
Created attachment 141185 [details] [review] Bug 31625: Reindent tools home and tools sidebar This patch performs general template cleanup to the tools-home and tools-menu templates: Make indentation consistent, replace tabs with spaces, fix invalid HTML. To test, apply the patch and go to Tools. The page should look correct, with no visible changes from before the patch. Go to any tools page which shows the left-hand sidebar, e.g. Patron lists, Batch item deletion, Inventory, etc. The menu should look correct. Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 141186 [details] [review] Bug 31625: (follow-up) Markup comments This patch adds comments to the templates to highlight the markup structure. This patch should have no effect on the pages' appearance or functionality. Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
(In reply to Katrin Fischer from comment #5) > Hm, I wonder if this does not clash with bug 31162 and if we should push one or > the other first?
It'll clash, but I'll have a quick look at rebasing the other ones upon this one.. hopefully it won't be too painful.
Pushed to master for 22.11. Nice work everyone, thanks!
Was about to say I rebased this morning so we're all good to push.. then I spotted Tomas already had.. hehe