On bug 38664 we are going to tidy the template file and so indent them properly. Most of our templates have the following structure: <div class="main container-fluid"> <div class="row"> <div class="col-md-10 order-md-2 order-sm-1"> <main> [% INCLUDE 'messages.inc' %] <!-- The content --> </main> </div> <div class="col-md-2 order-sm-2 order-md-1"> <aside> </aside> </div> </div> </div> It would be useful to have a WRAPPER to avoid: * errors (sometimes we see the two divs with the same order: sm-2) * inconsistencies * 4 indentation blocks (will be replaced with a single one)
This is how it would look like: https://gitlab.com/joubu/Koha/-/commit/086d8b126b918fe558699a2297514ea8ecb6adf3
This is how it would look like: https://gitlab.com/joubu/Koha/-/commit/360a3811f3194b762028c3a927c81f14322d6f38
This is ready for testing, use the commits from the branch of bug 38664.
Created attachment 177403 [details] [review] Bug 38718: (follow-up) Wrap both search form and search results in wrapper Also pass "wide_full" parameter to main-container.inc
Created attachment 177404 [details] [review] Bug 38718: (follow-up) Use main-container.inc on ILL requests page
Created attachment 177405 [details] [review] Bug 38718: (follow-up) Restore sidebar menu to point of sale page
Created attachment 177406 [details] [review] Bug 38718: (follow-up) Correct aside parameter
Created attachment 177407 [details] [review] Bug 38718: (follow-up) Use container wrapper on tags page