From 3da5337d8f36e4a8d65b920fc783969142ce3a48 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 1 Apr 2022 14:30:49 +0000 Subject: [PATCH] Bug 30434: Convert catalog merge page tabs to Bootstrap This patch updates the catalog merge template so that it uses Bootstrap tabs instead of jQueryUI. Some JavaScript is modified to accommodate the new DOM structure. The patch also makes a minor HTML correction, adding a missing "". To test, apply the patch and search the catalog in the staff client. - In the search results, check the checkbox for two records. - Choose Edit -> Merge records. - Click next to accept the merge reference selection. - On the next page you should see two tabs under "Source records." Confirm that they work correctly. - All checkboxes in the first tab should be checked by default. - Confirm that tag and subfield selection still works correctly. --- .../prog/en/modules/cataloguing/merge.tt | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt index 2f3864f29d..9c96be0823 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt @@ -31,6 +31,7 @@ div#result { margin-top: 1em; } Merging records +
@@ -289,13 +290,14 @@ div#result { margin-top: 1em; } rebuild_target($("#tabs"), $("#resultul")); }); - // Creating tabs - $("#tabs").tabs(); + if( $(".tab-pane.active").length < 1 ){ + $("#tabs a:first").tab("show"); + } - // Check all checkboxes in first tab, and uncheck all others to avoid - // inconsistencies from a page refresh. - $('#tabs div#tabrecord[% ref_biblionumber | html %]').find('input[type="checkbox"]').prop('checked', true); - $('#tabs > div:not("#tabrecord[% ref_biblionumber | html %]")').find('input[type="checkbox"]').prop('checked', false); + // Uncheck all checkboxes in both tabs, then check all checkboxes in first tab + // to avoid inconsistencies from a page refresh. + $("#tabs input[type='checkbox']").prop("checked", false); + $("#tabrecord[% ref_biblionumber | html %] input[type='checkbox']").prop("checked", true ); //Set focus to cataloging search $("input[name=q]:eq(0)").focus(); -- 2.20.1