From b011b7b4b075075010708d56f18834445576fbbb Mon Sep 17 00:00:00 2001
From: Dan Scott <dscott@laurentian.ca>
Date: Wed, 21 Sep 2011 21:56:39 -0400
Subject: [PATCH] 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>
---
 Open-ILS/src/templates/opac/results.tt2      | 12 ++++++------
 Open-ILS/web/css/skin/default/opac/style.css |  6 ------
 2 files changed, 6 insertions(+), 12 deletions(-)

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; }
-- 
2.11.0