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 acebea7..8876a50 100644 (file)
--- 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 e1536a6..9341d37 100644 (file)
--- 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; }