LP #1406350 Mobile Device Navigation Issue Fix for Shelf Browser
authorAdam Bowling <abowling@emeralddata.net>
Tue, 20 Jan 2015 21:54:11 +0000 (16:54 -0500)
committerBen Shum <bshum@biblio.org>
Tue, 3 Mar 2015 02:26:09 +0000 (21:26 -0500)
Currently, when a browser window is narrowed too much, whether
in a mobile browser, or on a desktop or laptop machine, the
"Previous Page" and "Next Page" navigation links disappear. The
only solution that seems to solve this is by removing the
"THEAD" element from the shelf browser table.

This patch removes that tag, adds a new row to imitate a new
border as existed before, and modifies the CSS in .../opac/
css/style.css.tt2 to address that difference.

Signed-off-by: Adam Bowling <abowling@emeralddata.net>
Signed-off-by: Yamil Suarez <yamil@yamil.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
Open-ILS/src/templates/opac/css/style.css.tt2
Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2

index 416e573..9c30244 100644 (file)
@@ -1200,8 +1200,10 @@ a.dash-link:hover { text-decoration: underline !important; }
 .expert-search-row { padding-top: 10px; }
 #adv_expert_row label { font-weight: bold; }
 
-.bookshelf thead tr td {
+.bookshelf tr.browse_border td {
     border-bottom: 1px dashed [% css_colors.accent_dark %];
+    font-size: 1px;
+    height: 1px;
     padding-bottom: 1ex;
 }
 .cn_browse_item { padding: 2ex; }
index 51ef1ad..835eab3 100644 (file)
@@ -9,7 +9,6 @@
             [% l("You are now browsing [_1]", ctx.browsing_ou.name) | html %]
         </p>
         <table class='data_grid bookshelf' width='100%'>
-            <thead>
                 <tr>
                     <td>
                         <a class='classic_link lbl1' 
@@ -21,7 +20,9 @@
                             href="[% mkurl('', {cnoffset => cnoffset + 1}) %]#cnbrowse">[% l("Next Page &gt;&gt;") %]</a>
                     </td>
                 </tr>
-            </thead>
+            <tr class='browse_border'>
+                <td colspan='3'>&nbsp;</td>
+            </tr>
             <tbody>
             [%- tr_open = 0;
                 PROCESS get_library;