Bug 20741

Summary: Update two-column templates with Bootstrap grid: Administration part 5
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, martin.renvoize, nick, testopia
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:
Circulation function:
Bug Depends on: 20668    
Bug Blocks: 20654    
Attachments: Bug 20741: Update two-column templates with Bootstrap grid: Administration part 4
Bug 20741: Update two-column templates with Bootstrap grid: Administration part 5
Bug 20741: Update two-column templates with Bootstrap grid: Administration part 5
Bug 20741: Update two-column templates with Bootstrap grid: Administration part 5

Description Owen Leonard 2018-05-09 12:02:41 UTC
This patch will convert still more administration templates to use the Bootstrap grid:

 - fieldmapping.tt
 - item_circulation_alerts.tt
 - itemtypes.tt
 - koha2marclinks.tt
 - localization.tt
 - marctagstructure.tt
Comment 1 Owen Leonard 2018-08-17 14:01:08 UTC Comment hidden (obsolete)
Comment 2 Claire Gravely 2018-08-21 07:03:32 UTC
Patch does not apply 

Using index info to reconstruct a base tree...
M	koha-tmpl/intranet-tmpl/prog/en/modules/admin/fieldmapping.tt
M	koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt
M	koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt
M	koha-tmpl/intranet-tmpl/prog/en/modules/admin/koha2marclinks.tt
M	koha-tmpl/intranet-tmpl/prog/en/modules/admin/localization.tt
M	koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
Falling back to patching base and 3-way merge...
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/localization.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/koha2marclinks.tt
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/admin/koha2marclinks.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/admin/item_circulation_alerts.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/fieldmapping.tt
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/admin/fieldmapping.tt
Failed to merge in the changes.
Comment 3 Katrin Fischer 2018-08-27 18:41:22 UTC
Owen, can you please rebase? Part 5 is missing from the sequence! :)
Comment 4 Owen Leonard 2018-08-29 12:31:57 UTC Comment hidden (obsolete)
Comment 5 Claire Gravely 2018-08-30 09:28:48 UTC Comment hidden (obsolete)
Comment 6 Katrin Fischer 2018-08-30 20:00:19 UTC
Created attachment 78316 [details] [review]
Bug 20741: Update two-column templates with Bootstrap grid: Administration part 5

This patch modifies several administration templates to use the
Bootstrap grid instead of YUI.

This patch also removes obsolete "text/javascript" attributes from
<script> tags in the modified templates.

To test, apply the patch and view the following pages, confirming that
they look correct at various browser widths:

 - Administration -> Keywords to MARC mapping
 - Administration -> Item circulation alerts
 - Administration -> Item types
   - View and edit item types
   - Edit and item type and translate a description
 - Administration -> Koha to MARC mapping
 - Administration -> MARC frameworks -> MARC structure
   - View and edit tags

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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Katrin Fischer 2018-08-30 20:01:00 UTC
Missing the table-wise filters on koha2marclinks, that's entirely unrelated to this patch (I checked ;) )
Comment 8 Nick Clemens (kidclamp) 2018-09-06 17:40:41 UTC
Awesome work all! 

Pushed to master for 18.11
Comment 9 Martin Renvoize (ashimema) 2018-09-10 17:23:10 UTC
Enhancement, will not be backported to 18.05.x series.