From: Chris Sharp <csharp@georgialibraries.org>
Date: Tue, 22 Nov 2016 13:54:26 +0000 (-0500)
Subject: OPAC advanced search page UI tweaks. (Terran)
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=099a674f4107c7d6b33de73716936eae4ae01588;p=evergreen%2Fpines.git

OPAC advanced search page UI tweaks. (Terran)
---

diff --git a/Open-ILS/src/templates/opac/parts/advanced/search.tt2 b/Open-ILS/src/templates/opac/parts/advanced/search.tt2
index dc650a1c7b..cfc9d7fe26 100644
--- a/Open-ILS/src/templates/opac/parts/advanced/search.tt2
+++ b/Open-ILS/src/templates/opac/parts/advanced/search.tt2
@@ -12,7 +12,7 @@
                     <tr>
                         <td>
                             <div class="header_middle" id="adv_search_input">
-                                [% l("Search Input") %]
+                                Enter Search Words:
                             </div>
                         </td>
                     </tr>
@@ -22,7 +22,8 @@
                     <!-- add a new row -->
                     <tr id='adv_global_addrow'>
                         <td align='left'>
-                            <a href="javascript:;" id="myopac_new_global_row" onclick='addSearchRow();'>[% l('Add Search Row') %]</a>
+                            <a href="javascript:;" id="myopac_new_global_row" onclick='addSearchRow();'>Add Another Search Row</a>
+                            <br/> <br/>
                         </td>
                     </tr>
                 </tbody>
@@ -30,24 +31,10 @@
         </div>
     </div>
 
-    <div id='adv_search_submit'>
-        <input type="hidden" name="_adv" value="1" />
-        <input id="detail" type="hidden" name="detail_record_view"
-               value="[% show_detail_view %]"/>
-        <span>
-            <input id='search-submit-go' type="submit" value="[% l('Search') %]" title="[% l('Search') %]" class="opac-button"
-                onclick='setTimeout(function(){$("search-submit-spinner").className=""; $("search-submit-go").className="hidden"}, 2000)'/>
-            <img id='search-submit-spinner' src='/opac/images/progressbar_green.gif'
-                class='hidden' alt="[% l('Search in progress icon') %]"/>
-        </span>
-
-        <a href="[% mkurl(ctx.opac_root _ '/advanced', {$loc_name => loc_value}, 1) %]"
-            class="opac-button">[% l('Clear Form') %]</a>
-    </div>
 
     <div id='adv_search_filters'>
-        <div class="header_middle">[% l('Search Filters') %]</div>
-
+        <div class="header_middle">Choose Search Limits:</div>
+        <p style="font-style:italic;margin-bottom:0px;">Tip: Ctrl-Click to select multiple lines or to deselect lines (Shift-Click for Macs).</p>
 [%
     in_row = 0;
     FOR adv_chunk IN search.adv_config;
@@ -69,13 +56,7 @@
                 CASE "lib_selector";
                     PROCESS "opac/parts/org_selector.tt2";
                         INCLUDE build_org_selector show_loc_groups=1 id=adv_chunk.id %]
-                            <div class="adv_search_available">
-                                <input type='checkbox' name="modifier"
-                                    value="available"[% CGI.param('modifier').grep('available').size ? ' checked="checked"' : '' %]
-                                    id='opac.result.limit2avail' />
-                                <label for='opac.result.limit2avail'>
-                                    [% l("Limit to Available") %]</label>
-                            </div>
+
 [%
                 CASE "pub_year"; %]
                             <select name='pubdate'
@@ -127,23 +108,11 @@
                         value=default_sort class='results_header_sel';
                     %]
 
-                    [% IF NOT metarecords.disabled %]
-                        <br/><!-- <br> may seem redundant, but it allows the
-                            <input> (below) to drop down inline w/ its label -->
-                        <div class="adv_search_available">
-                            <input type='checkbox' name="modifier" value="metabib"
-                              [%- CGI.param('modifier').grep('metabib').size ?
-                                    ' checked="checked"' : '' %]
-                                id='opac.result.ismetabib' />
-                            <label for='opac.result.ismetabib'>
-                                [% l("Group Formats and Editions") %]</label>
-                        </div>
-                    [% END %]
 
                 [% CASE "copy_location" %]
                     <select id="adv_copy_location_selector"
                         title="[% l('Select Shelving Location') %]"
-                        name="fi:locations" size="3" multiple="multiple">
+                        name="fi:locations" size="4" multiple="multiple">
                     </select>
                 [%
             END;
@@ -170,5 +139,43 @@
 
         </div>
     </div>
+    
+    <!-- moved checkboxes into their own div for readability -->
+    <div style="clear:left;padding-left:20px;">
+        <input type='checkbox' name="modifier"
+            value="available"[% CGI.param('modifier').grep('available').size ? ' checked="checked"' : '' %]
+            id='opac.result.limit2avail' />
+        <label for='opac.result.limit2avail'>
+             <strong>[% l("Limit to Available") %]</strong>
+        </label>
+        <br/>
+        [% IF NOT metarecords.disabled %]
+            <input type='checkbox' name="modifier" value="metabib"
+                [%- CGI.param('modifier').grep('metabib').size ?
+                    ' checked="checked"' : '' %]
+                    id='opac.result.ismetabib' />
+            <label for='opac.result.ismetabib'>
+                <strong>[% l("Group Formats and Editions") %]</strong>
+            </label>
+        [% END %]
+    </div>
+    
+    <!-- moved search buttons to bottom -->
+    <div id='adv_search_submit' style="clear:both;text-align:center;width:100%;max-width:700px;">
+        <input type="hidden" name="_adv" value="1" />
+        <input id="detail" type="hidden" name="detail_record_view"
+               value="[% show_detail_view %]"/>
+               
+        <a href="[% mkurl(ctx.opac_root _ '/advanced', {$loc_name => loc_value}, 1) %]"
+            class="opac-button" style="padding:10px;background-color:#669999;">[% l('Clear Form') %]</a>
+            
+        <span>
+            <input id='search-submit-go' type="submit" value="[% l('Search') %]" title="[% l('Search') %]" class="opac-button"
+                onclick='setTimeout(function(){$("search-submit-spinner").className=""; $("search-submit-go").className="hidden"}, 2000)' 
+                style="padding:10px 50px 10px 50px;" />
+            <img id='search-submit-spinner' src='/opac/images/progressbar_green.gif'
+                class='hidden' alt="[% l('Search in progress icon') %]"/>
+        </span>
+    </div>
 </div>
 </form>