From: Dan Scott <dscott@laurentian.ca>
Date: Thu, 22 Sep 2011 01:56:39 +0000 (-0400)
Subject: Group "Sort by" and sort select box together
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=b011b7b4b075075010708d56f18834445576fbbb;p=evergreen%2Fpines.git

Group "Sort by" and sort select box together

The sort select box was weirdly way over on the right. Fix that.

Also address slightly weird vertical alignment of some elements of the
results header bar.

Also use <label> to group the "sort by" with the select box; a bit
better for screen reasers.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
---

diff --git a/Open-ILS/src/templates/opac/results.tt2 b/Open-ILS/src/templates/opac/results.tt2
index acebea7d3e..8876a505a8 100644
--- a/Open-ILS/src/templates/opac/results.tt2
+++ b/Open-ILS/src/templates/opac/results.tt2
@@ -36,8 +36,9 @@
                 </div>
                 [% END %]
                 <div class="results_header_div"></div>
-                    <div class="results_header_lbl">[% l('Sort by') %]</div>
+                    <label class="results_header_lbl">[% l('Sort by') %]
                     [% INCLUDE "opac/parts/filtersort.tt2" value=CGI.param('sort') submit_on_change=1 %]
+                    </label>
                     <div class="results_header_div"></div>
 
                     <div class='results_header_sel' id='simple-detail-view-links'>
@@ -49,11 +50,10 @@
                     </div>
                     <div class="results_header_div"></div>
 
-                    <input type="checkbox" id="limit_to_available" name="modifier" value="available"
-                        onchange="limit_to_avail_onchange(this, true)"
-                        [% CGI.param('modifier').grep('available').size ? ' checked="checked"' : '' %] />
-
-                    <label for="limit_to_available" class="results_header_lbl">
+                    <label class="results_header_lbl">
+                        <input type="checkbox" id="limit_to_available" name="modifier" value="available"
+                            onchange="limit_to_avail_onchange(this, true)"
+                            [% CGI.param('modifier').grep('available').size ? ' checked="checked"' : '' %] />
                         [% l('Limit to available items') %]
                     </label>
                 <div class="clear-both"></div>
diff --git a/Open-ILS/web/css/skin/default/opac/style.css b/Open-ILS/web/css/skin/default/opac/style.css
index e1536a6b9e..9341d376af 100644
--- a/Open-ILS/web/css/skin/default/opac/style.css
+++ b/Open-ILS/web/css/skin/default/opac/style.css
@@ -630,8 +630,6 @@ div.select-wrapper:hover {
 	float: left;
 	font-size: 11px;
 	color: #191919;
-	position: relative;
-	top: 5px;
 	margin-right: 6px;
 }
 
@@ -920,10 +918,6 @@ div.select-wrapper:hover {
 #adv_global_input_table select { width: 7em; }
 .adv_adv_link { font-size: 8pt; color: red; }
 #acct_prefs_header { float: left; }
-#limit_to_available {
-    float: left; position: relative; top: 2px;
-    left: -2px; margin-right: 4px;
-}
 .search_page_nav_link { cursor: pointer; }
 #opac.result.sort { width: 160px; }
 .renew-summary { font-size: 125%; font-style: italic; margin: 0.5ex 0; }