From e8e9e64b774b8baa707e40ce3fa26c0a96c85bb3 Mon Sep 17 00:00:00 2001
From: Adam Bowling <abowling@emeralddata.net>
Date: Tue, 20 Jan 2015 16:54:11 -0500
Subject: [PATCH] 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>
---
 Open-ILS/src/templates/opac/css/style.css.tt2      |  4 +++-
 .../src/templates/opac/parts/record/cnbrowse.tt2   | 25 +++++++++++-----------
 2 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 416e573581..9c30244a22 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -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; }
diff --git a/Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2 b/Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2
index 51ef1adf4b..f735a7591e 100644
--- a/Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/cnbrowse.tt2
@@ -9,19 +9,20 @@
             [% 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' 
+            <tr>
+                <td>
+                    <a class='classic_link lbl1' 
                             href="[% mkurl('', {cnoffset => cnoffset - 1}) %]#cnbrowse">[% l("&lt;&lt; Previous Page") %]</a>
-                    </td>
-                    <td colspan='1' align='center'>[% l("Shelf Browser") %]</td>
-                    <td>
-                        <a class='classic_link lbl1' 
-                            href="[% mkurl('', {cnoffset => cnoffset + 1}) %]#cnbrowse">[% l("Next Page &gt;&gt;") %]</a>
-                    </td>
-                </tr>
-            </thead>
+                </td>
+                <td colspan='1' align='center'>[% l("Shelf Browser") %]</td>
+                <td>
+                    <a class='classic_link lbl1' 
+                        href="[% mkurl('', {cnoffset => cnoffset + 1}) %]#cnbrowse">[% l("Next Page &gt;&gt;") %]</a>
+                </td>
+            </tr>
+            <tr class='browse_border'>
+                <td colspan='3'>&nbsp;</td>
+            </tr>
             <tbody>
             [%- tr_open = 0;
                 PROCESS get_library;
-- 
2.11.0