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:25:57 +0000 (21:25 -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 f93bf9e..9a2d2f6 100644 (file)
@@ -1192,8 +1192,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 4699883..2dba5e5 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" rel="nofollow" vocab="">[% 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;