Bug 33278: Correct JS for activating default tab on various pages
The conversion of the header search tabs to Bootstrap resulted in a bug on several staff interface pages: The first tab isn't selected by default. The jQuery selector isn't specific enough. The patch also corrects a template error which was introduced to merge-record.inc by Bug 32683. To test, apply the patch and test the following pages to confirm that the first tab on each page is selected by default: - Catalog -> Advanced search - Catalog -> MARC details - Catalog -> Search results -> Merge - Administration -> Authorized values -> Edit - Administratoin -> Library transfer limits - Administration -> Item types -> Edit - Authorities -> Authority details - Authorities -> Search -> Merge Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
parent
63772a9cb2
commit
df6432fcc8
|
|
@ -48,7 +48,6 @@
|
|||
<div id="tabs" class="toptabs">
|
||||
<h2>Source records</h2>
|
||||
<ul class="nav nav-tabs" role="tablist">
|
||||
<div class="tab-content">
|
||||
[% FOREACH record IN sourcerecords %]
|
||||
<li role="presentation">
|
||||
<a href="#tabrecord[% record.recordid | uri %]" aria-controls="tabrecord[% record.recordid | uri %]" role="tab" data-toggle="tab">
|
||||
|
|
|
|||
|
|
@ -464,7 +464,7 @@
|
|||
return confirm(_("Are you sure you want to delete this authorized value category?"));
|
||||
});
|
||||
|
||||
if( $(".tab-pane.active").length < 1 ){
|
||||
if( $("#icons .tab-pane.active").length < 1 ){
|
||||
$("#icons a:first").tab("show");
|
||||
}
|
||||
});
|
||||
|
|
|
|||
|
|
@ -136,7 +136,7 @@
|
|||
});
|
||||
});
|
||||
|
||||
if( $(".tab-pane.active").length < 1 ){
|
||||
if( $("#transferlimit_tabs .tab-pane.active").length < 1 ){
|
||||
$("#transferlimit_tabs a:first").tab("show");
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -643,7 +643,7 @@ Item types › Administration › Koha
|
|||
toUC(this);
|
||||
});
|
||||
$(".library_limitation").tooltip();
|
||||
if( $(".tab-pane.active").length < 1 ){
|
||||
if( $("#icons .tab-pane.active").length < 1 ){
|
||||
$("#icons a:first").tab("show");
|
||||
}
|
||||
});
|
||||
|
|
|
|||
|
|
@ -136,7 +136,7 @@
|
|||
[% END %]
|
||||
<script>
|
||||
$(document).ready(function() {
|
||||
if( $(".tab-pane.active").length < 1 ){
|
||||
if( $("#authoritiestabs .tab-pane.active").length < 1 ){
|
||||
$("#authoritiestabs a:first").tab("show");
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -149,7 +149,7 @@ div#result { margin-top: 1em; }
|
|||
ev.preventDefault();
|
||||
newin=window.open($(this).attr('href'),'merge_reference', 'width=1000,height=600,toolbar=false,scrollbars=yes');
|
||||
});
|
||||
if( $(".tab-pane.active").length < 1 ){
|
||||
if( $("#tabs .tab-pane.active").length < 1 ){
|
||||
$("#tabs a:first").tab("show");
|
||||
}
|
||||
});
|
||||
|
|
|
|||
|
|
@ -232,7 +232,7 @@
|
|||
$("#Frameworks").on("change",function(){
|
||||
Changefwk(this);
|
||||
});
|
||||
if( $(".tab-pane.active").length < 1 ){
|
||||
if( $("#bibliotabs .tab-pane.active").length < 1 ){
|
||||
$("#bibliotabs a:first").tab("show");
|
||||
}
|
||||
});
|
||||
|
|
|
|||
|
|
@ -419,7 +419,7 @@
|
|||
var Sticky;
|
||||
$(document).ready(function() {
|
||||
$("input[name=q]:eq(0)").focus();
|
||||
if( $(".tab-pane.active").length < 1 ){
|
||||
if( $("#advsearches .tab-pane.active").length < 1 ){
|
||||
$("#advsearches a:first").tab("show");
|
||||
}
|
||||
Sticky = $("#toolbar");
|
||||
|
|
|
|||
|
|
@ -293,7 +293,7 @@ div#result { margin-top: 1em; }
|
|||
rebuild_target($("#tabs"), $("#resultul"));
|
||||
});
|
||||
|
||||
if( $(".tab-pane.active").length < 1 ){
|
||||
if( $("#tabs .tab-pane.active").length < 1 ){
|
||||
$("#tabs a:first").tab("show");
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue