Bug 32914: Use template wrapper for batch record deletion and modification templates
authorOwen Leonard <oleonard@myacpl.org>
Wed, 8 Feb 2023 17:46:16 +0000 (17:46 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Tue, 16 May 2023 18:17:32 +0000 (15:17 -0300)
commit3c2eb1da53dcdb6eeb5f85da2716da9b6ec022e4
tree8fa8345f0564eb075bdabbd1c07d729f615939bc
parent8b1a5e34a7a441ac4783f22a7bfd0655fa302200
Bug 32914: Use template wrapper for batch record deletion and modification templates

This patch updates the batch record modification and batch record
deletion templates to replace tab markup with the use of WRAPPERs.

This patch adds a "clearfix" snippet of CSS to _mixins.scss copied from
Bootstrap. Applied to the active tab pane, this helps in situations
where Bootstrap tabs follow a floated element.

The patch also wraps tab label strings in <span> to ensure they can be
translated.

Unrelated: The patch removes an extra </option> from the batch record
modification template.

To test you should have at least one list and at least one MARC
modification template defined.

- Apply the patch and go to Cataloging -> Batch record modification
- You should see three tabs, "Upload a file," "Select a list of
  records," and "Enter a list of record numbers."
- The tabs should look correct and work correctly.
- Checking the "Authorities" radio button should hide the lists tab.
- Test that submissions from each form work correctly.

- Go to Cataloging -> Batch record deletion and perform the same tests.

Signed-off-by: Philip Orr <philip.orr@lmscloud.de>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/css/src/_mixins.scss
koha-tmpl/intranet-tmpl/prog/css/src/staff-global.scss
koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_delete_records.tt
koha-tmpl/intranet-tmpl/prog/en/modules/tools/batch_record_modification.tt