From e6227af570de4ec3726fcdb8dce3059ec1ef0d6f Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Tue, 2 Jun 2020 14:00:25 +0000 Subject: [PATCH] Bug 25653: Update tab selector to use 'active' and include all options It looks like the functionality was broken when ui tabs were upgraded and mainmainentry was added To test: 1 - Browse to Authorities 2 - Search in each tab 3 - Note all search results revert to 'Search main heading a only' 4 - Apply patch 5 - Note selection is retained after search Signed-off-by: Claire Gravely Signed-off-by: Julian Maurice Signed-off-by: Jonathan Druart (cherry picked from commit 4365b64403ba9492650e2e1733ccad8772dcdfc2) Signed-off-by: Lucas Gass (cherry picked from commit e77736c6d6683d145c7f66821c70a24c4e07e2f8) Signed-off-by: Aleisha Amohia --- .../intranet-tmpl/prog/en/includes/authorities_js.inc | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities_js.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities_js.inc index f2966019cf..555b8da0dc 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities_js.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities_js.inc @@ -77,12 +77,14 @@ $(document).ready(function () { var searchType = '[% marclist | html %]'; if (searchType) { - if ('mainentry' == searchType) { - $("#header_search").tabs( "option", "selected", 0 ); + if ('mainmainentry' == searchType) { + $("#header_search").tabs( "option", "active", 0 ); + } else if ('mainentry' == searchType) { + $("#header_search").tabs( "option", "active", 1 ); } else if ('match' == searchType) { - $("#header_search").tabs( "option", "selected", 1 ); + $("#header_search").tabs( "option", "active", 2 ); } else if ('all' == searchType) { - $("#header_search").tabs( "option", "selected", 2 ); + $("#header_search").tabs( "option", "active", 3 ); } } -- 2.39.5