From: Adam Bowling <abowling@emeralddata.net>
Date: Tue, 20 Jan 2015 21:54:11 +0000 (-0500)
Subject: LP #1406350 Mobile Device Navigation Issue Fix for Shelf Browser
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=0f53eb85e5ad98d19874c304d55034c8b524158f;p=working%2FEvergreen.git

LP #1406350 Mobile Device Navigation Issue Fix for Shelf Browser

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>
---

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index f93bf9e635..9a2d2f626f 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -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; }
diff --git a/Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2 b/Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2
index 4699883a4a..2dba5e54cd 100644
--- a/Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2
@@ -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;