Bug 20037

Summary: Switch single-column templates to Bootstrap grid: Serials
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: claire_gravely, nick, testopia
Version: MainKeywords: Academy
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:
Bug Depends on:    
Bug Blocks: 19980    
Attachments: Bug 20037: Switch single-column templates to Bootstrap grid: Serials
Bug 20037: Switch single-column templates to Bootstrap grid: Serials
Bug 20037: Switch single-column templates to Bootstrap grid: Serials
Bug 20037: Switch single-column templates to Bootstrap grid: Serials

Description Owen Leonard 2018-01-18 18:31:51 UTC
There are several single-column Circulation templates which can be updated to use the Bootstrap grid.

- serials\acqui-search-result.tt
- serials\acqui-search.tt
- serials\result.tt
- serials\subscription-bib-search.tt
Comment 1 Owen Leonard 2018-01-18 19:21:31 UTC Comment hidden (obsolete)
Comment 2 Claire Gravely 2018-01-24 07:20:10 UTC
hiya,
acqui-search.tt is mising the bootstrap grid.
Comment 3 Claire Gravely 2018-01-24 12:14:35 UTC
(In reply to Claire Gravely from comment #2)
> hiya,
> acqui-search.tt is mising the bootstrap grid.

Ignore that. Everything is lovely!
Comment 4 Claire Gravely 2018-01-24 12:19:32 UTC
(In reply to Claire Gravely from comment #3)
> (In reply to Claire Gravely from comment #2)
> > hiya,
> > acqui-search.tt is mising the bootstrap grid.
> 
> Ignore that. Everything is lovely!

Sorry, double checked the wrong bit :| It does seem to be missing.
Comment 5 Owen Leonard 2018-02-06 19:09:55 UTC
Created attachment 71269 [details] [review]
Bug 20037: Switch single-column templates to Bootstrap grid: Serials

This patch updates several single-column Serials module templates to
use the Bootstrap grid. In addition to grid changes, some templates have
been modified to include the footer with the correct popup parameter.

- serials\acqui-search.tt - Go to Serials -> New subscription. Click
  "Search for a vendor."

- serials\acqui-search-result.tt - Vendor search results

- serials\result.tt - Click "Search for record."

- serials\subscription-bib-search.tt - Catalog search results.

Each of these pages should look correct.
Comment 6 Owen Leonard 2018-02-06 19:20:11 UTC
I submitted a follow-up to make this patch more consistent, but: It's not that acqui-search.tt was missing the Bootstrap grid, it's that it has minimal Bootstrap markup because it's a very simple template. It's not really necessary to define columns and rows when there aren't multiples of either.
Comment 7 Claire Gravely 2018-02-08 14:36:31 UTC
Created attachment 71350 [details] [review]
Bug 20037: Switch single-column templates to Bootstrap grid: Serials

This patch updates several single-column Serials module templates to
use the Bootstrap grid. In addition to grid changes, some templates have
been modified to include the footer with the correct popup parameter.

- serials\acqui-search.tt - Go to Serials -> New subscription. Click
  "Search for a vendor."

- serials\acqui-search-result.tt - Vendor search results

- serials\result.tt - Click "Search for record."

- serials\subscription-bib-search.tt - Catalog search results.

Each of these pages should look correct.

Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>
Comment 8 Katrin Fischer 2018-02-18 14:04:11 UTC
Created attachment 71908 [details] [review]
Bug 20037: Switch single-column templates to Bootstrap grid: Serials

This patch updates several single-column Serials module templates to
use the Bootstrap grid. In addition to grid changes, some templates have
been modified to include the footer with the correct popup parameter.

- serials\acqui-search.tt - Go to Serials -> New subscription. Click
  "Search for a vendor."

- serials\acqui-search-result.tt - Vendor search results

- serials\result.tt - Click "Search for record."

- serials\subscription-bib-search.tt - Catalog search results.

Each of these pages should look correct.

Signed-off-by: Claire Gravely <claire.gravely@bsz-bw.de>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 9 Jonathan Druart 2018-02-18 19:50:33 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 10 Nick Clemens 2018-03-23 00:59:22 UTC
Enhancement, not backported for 17.11