[Bug 42330] New: Use main-container.inc from all the templates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Bug ID: 42330 Summary: Use main-container.inc from all the templates Initiative type: --- Sponsorship --- status: Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Staff interface Assignee: jonathan.druart@gmail.com Reporter: jonathan.druart@gmail.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com We can now use main-container.inc with a custom aside (see bug 42322). Now we need to use it for all our templates. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |42322 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42322 [Bug 42322] Collapse sidebar -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |oleonard@myacpl.org Depends on| |42094 --- Comment #1 from Jonathan Druart <jonathan.druart@gmail.com> --- I am ready to work on this as soon as the depend bug reports are ready to be pushed. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42094 [Bug 42094] Tidy all script tags (except value_builder) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #2 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 197164 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197164&action=edit Bug 42330: Use main-container from all our templates This goal here is to uniformize the structure of our templates. Using main-container.inc will also introduce the sidebar collapse (from bug 42322) where it was missing -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #3 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 197165 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197165&action=edit Bug 42330: Tidy Patch from commit cfefca4 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |michaela.sieber@kit.edu -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197164|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197165|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #4 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 197166 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197166&action=edit Bug 42330: Use main-container from all our templates This goal here is to uniformize the structure of our templates. Using main-container.inc will also introduce the sidebar collapse (from bug 42322) where it was missing -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #5 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 197167 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197167&action=edit Bug 42330: Tidy Patch from commit 2b316ca -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |42344 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42344 [Bug 42344] Remove CircSidebar -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff --- Comment #6 from Jonathan Druart <jonathan.druart@gmail.com> --- Test plan: Check the views of the following templates and confirm that the sidebar can now be collapsed includes/facets.inc (the facets of the search) acqui/histsearch.tt acqui/invoices.tt acqui/lateorders.tt acqui/parcels.tt admin/aqbudgets.tt admin/aqplan.tt admin/float_limits.tt admin/sru_modmapping.tt sru_modmapping_auth.tt cataloguing/editor.tt circ/overdue.tt circ/pendingbookings.tt circ/pendingreserves.tt circ/reserveratios.tt circ/view_holdsqueue.tt members/member.tt members/memberentrygen.tt plugins/plugins-home.tt serials/serials-edit.tt tags/list.tt tools/additional-contents.tt tools/showdiffmarc.tt tools/stockrotation.tt tools/upload.tt -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #7 from Owen Leonard <oleonard@myacpl.org> --- [FAIL] koha-tmpl/intranet-tmpl/prog/en/includes/main-container.inc FAIL filters missing_filter at line 12 ([% messages_block %]) missing_filter at line 26 ([% aside_block %]) missing_filter at line 78 ([% messages_block %]) missing_filter at line 88 ([% messages_block %]) missing_filter at line 98 ([% messages_block %]) Do these all need "| $raw" ? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197166|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197167|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #8 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 197365 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197365&action=edit Bug 42330: Use main-container from all our templates This goal here is to uniformize the structure of our templates. Using main-container.inc will also introduce the sidebar collapse (from bug 42322) where it was missing -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #9 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 197366 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197366&action=edit Bug 42330: Tidy Patch from commit 1189aa3 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #10 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to Owen Leonard from comment #7)
[FAIL] koha-tmpl/intranet-tmpl/prog/en/includes/main-container.inc FAIL filters missing_filter at line 12 ([% messages_block %]) missing_filter at line 26 ([% aside_block %]) missing_filter at line 78 ([% messages_block %]) missing_filter at line 88 ([% messages_block %]) missing_filter at line 98 ([% messages_block %])
Do these all need "| $raw" ?
Yes, fixed! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197365|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197366|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #11 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 197381 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197381&action=edit Bug 42330: Use main-container from all our templates This goal here is to uniformize the structure of our templates. Using main-container.inc will also introduce the sidebar collapse (from bug 42322) where it was missing Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #12 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 197382 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197382&action=edit Bug 42330: Tidy Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #13 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 197383 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197383&action=edit Bug 42330: (follow-up) Some corrections to sidebars - The structure of the search facets, advanced MARC editor, and HTML customiztions sidebars need to be corrected in order for the collapse button to work correctly. - The "toggle_sidebar" click handler should be inside a "$(document).ready()" in order to work correctly on pages where the JS is in the header. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197383|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #14 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 197388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197388&action=edit Bug 42330: (follow-up) Some corrections to sidebars - The structure of the search facets, advanced MARC editor, and HTML customiztions sidebars need to be corrected in order for the collapse button to work correctly. - The "toggle_sidebar" click handler should be inside a "$(document).ready()" in order to work correctly on pages where the JS is in the header. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #197381|0 |1 is obsolete| | Attachment #197382|0 |1 is obsolete| | Attachment #197388|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #15 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200057 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200057&action=edit Bug 42330: Use main-container from all our templates This goal here is to uniformize the structure of our templates. Using main-container.inc will also introduce the sidebar collapse (from bug 42322) where it was missing Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #16 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200058 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200058&action=edit Bug 42330: Tidy Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #17 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200059 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200059&action=edit Bug 42330: (follow-up) Some corrections to sidebars - The structure of the search facets, advanced MARC editor, and HTML customiztions sidebars need to be corrected in order for the collapse button to work correctly. - The "toggle_sidebar" click handler should be inside a "$(document).ready()" in order to work correctly on pages where the JS is in the header. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on|42094 | --- Comment #18 from Jonathan Druart <jonathan.druart@gmail.com> --- Patches rebased and removed the dependency on bug 42094 to ease adoption. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42094 [Bug 42094] Tidy all script tags (except value_builder) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #19 from David Nind <david@davidnind.com> --- Patches from dependent bug 42322 don't apply cleanly 8-(... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #200057|0 |1 is obsolete| | Attachment #200058|0 |1 is obsolete| | Attachment #200059|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #20 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200513 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200513&action=edit Bug 42330: Use main-container from all our templates This goal here is to uniformize the structure of our templates. Using main-container.inc will also introduce the sidebar collapse (from bug 42322) where it was missing Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #21 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200514 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200514&action=edit Bug 42330: Tidy Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #22 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200515 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200515&action=edit Bug 42330: (follow-up) Some corrections to sidebars - The structure of the search facets, advanced MARC editor, and HTML customiztions sidebars need to be corrected in order for the collapse button to work correctly. - The "toggle_sidebar" click handler should be inside a "$(document).ready()" in order to work correctly on pages where the JS is in the header. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #23 from Jonathan Druart <jonathan.druart@gmail.com> --- Patches rebased. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #24 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200516 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200516&action=edit Bug 42330: Add an xt test to prevent new occurrences of <main> in staff templates Patch from commit 1789636 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #25 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200517 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200517&action=edit Bug 42330: Add new template tools/restore-records.tt Patch from commit baad327 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #26 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200518 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200518&action=edit Bug 42330: Auto-tidy Patch from commit 3ccaa12 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply --- Comment #27 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Currently doesn't apply clean to main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #200514|0 |1 is obsolete| | Attachment #200515|0 |1 is obsolete| | Attachment #200516|0 |1 is obsolete| | Attachment #200517|0 |1 is obsolete| | Attachment #200518|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #28 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201039 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201039&action=edit Bug 42330: Tidy Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #29 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201040 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201040&action=edit Bug 42330: (follow-up) Some corrections to sidebars - The structure of the search facets, advanced MARC editor, and HTML customiztions sidebars need to be corrected in order for the collapse button to work correctly. - The "toggle_sidebar" click handler should be inside a "$(document).ready()" in order to work correctly on pages where the JS is in the header. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #30 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201041 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201041&action=edit Bug 42330: Add an xt test to prevent new occurrences of <main> in staff templates Patch from commit 2a75719 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #31 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201042 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201042&action=edit Bug 42330: Add new template tools/restore-records.tt Patch from commit 0e0ae59 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 --- Comment #32 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201043 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201043&action=edit Bug 42330: Auto-tidy Patch from commit ba3f1c1 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lisette@bywatersolutions.co | |m QA Contact|testopia@bugs.koha-communit |lisette@bywatersolutions.co |y.org |m -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |BLOCKED --- Comment #33 from Lisette Scheer <lisette@bywatersolutions.com> --- I can't apply the dependancy, so blocked for now. Details on $2322 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42330 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org