From 2d877b93d6d3bf479e7a089f7aeff6ee5cdf4da3 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 26 May 2015 12:10:50 +0200 Subject: [PATCH] Bug 14189: Use keep_text for the authority header includes/authorities-search.inc already used the same behavior but with another method. For consistency it's better to have only 1 way to do that. Test plan: Go on the authority home page Fill the input in the header and click on tab 'search all headings' and 'keyword search' and confirm that the text you have filled is kept. I tested after applying the patch and the text is kept after removing the part from authorities-search.inc. I also tried applying that jQuery solution instead of my JS loop but could not get it to work for the other pages with a reasonable ammount of effort. Signed-off-by: Jonathan Druart Signed-off-by: Eivin Giske Skaaren Signed-off-by: Tomas Cohen Arazi --- .../prog/en/includes/authorities-search.inc | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search.inc index 433c92703d..d7ce46dc0d 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search.inc @@ -14,11 +14,6 @@ $("#header_search").tabs( "option", "selected", 2 ); } } - $("#header_search input[type='text'], #header_search select").change(function() { - var myname = $(this).attr("name"); - var myval = $(this).val(); - $("#header_search [name='" + myname + "']").val(myval); - }); }); //]]> @@ -58,7 +53,7 @@ [% END %] - + - + - +