From 9719284622e2c3027ef615286ef417f920238c9f Mon Sep 17 00:00:00 2001
From: Josef Moravec <josef.moravec@gmail.com>
Date: Tue, 9 May 2017 13:33:52 +0000
Subject: [PATCH] Bug 18551: followup - hide advanced filters in header, move
 hidding to css file
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Test plan:
The same as first patch, but also with advanced search form in header hidden
on page load - see comment 4

Issue with advanced search form is gone.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Philippe <philippe.audet-fortin@inlibro.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
---
 koha-tmpl/intranet-tmpl/prog/css/staff-global.css        | 6 ++++++
 .../intranet-tmpl/prog/en/includes/patron-search.inc     | 9 ++++++---
 .../intranet-tmpl/prog/en/modules/members/member.tt      | 4 ++--
 3 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css
index fc5a854d9d..fb09d5131a 100644
--- a/koha-tmpl/intranet-tmpl/prog/css/staff-global.css
+++ b/koha-tmpl/intranet-tmpl/prog/css/staff-global.css
@@ -3001,3 +3001,9 @@ fieldset.rows + fieldset.action {
 .yui-u .rows li p label.widelabel {
     width: auto;
 }
+
+#pat_member #patron_list_dialog,
+#pat_member #searchresults,
+#patron_search #filters {
+    display: none;
+}
diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc
index 337f550590..ad82db303c 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc
@@ -162,9 +162,12 @@ $(document).ready(function() {
         $('#filters').toggle();
         $('.filteraction').toggle();
     });
-    [% IF ( advsearch ) %]$("#filteraction_on").toggle();
-    [% ELSE %]$("#filters").toggle();
-    $("#filteraction_off").toggle();[% END %]
+    [% IF ( advsearch ) %]
+        $("#filteraction_on").toggle();
+        $("#filters").show();
+    [% ELSE %]
+        $("#filteraction_off").toggle();
+    [% END %]
     [% SET dateformat = Koha.Preference('dateformat') %]
     $("#searchfieldstype").change(function() {
       if ( $(this).val() == 'dateofbirth' ) {
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt
index 7c8c9602c9..f8f2471bd7 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt
@@ -328,7 +328,7 @@ function filterByFirstLetterSurname(letter) {
       <div class="yui-b">
         <div class="yui-g">
           [% IF CAN_user_tools_manage_patron_lists %]
-            <div id="patron_list_dialog" class="dialog alert" style="display:none">
+            <div id="patron_list_dialog" class="dialog alert">
               Added <span class="patrons-length"></span> patrons to <a></a>.
             </div>
           [% END %]
@@ -355,7 +355,7 @@ function filterByFirstLetterSurname(letter) {
             </div>
           [% END %]
 
-          <div id="searchresults" style="display:none">
+          <div id="searchresults">
             <div id="searchheader">
               <h3>Patrons found for: <span id="searchpattern">[% IF searchmember %] for '[% searchmember %]'[% END %]</span></h3>
             </div>
-- 
2.39.5