From 938f11eb5b2652fc197eb628d4a52a449276fb69 Mon Sep 17 00:00:00 2001
From: Dan Scott <dscott@laurentian.ca>
Date: Mon, 23 Sep 2013 22:44:18 -0400
Subject: [PATCH] TPAC: Avoid paginator HTML rather than hiding it

Rather than generating HTML and then hiding it via CSS, just don't
generate the HTML at all. In this case, avoid generating the Previous /
Next links in those cases where we're on either page one or the last page
of results respectively.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
---
 Open-ILS/src/templates/opac/css/style.css.tt2         | 10 +++++-----
 Open-ILS/src/templates/opac/parts/result/paginate.tt2 | 16 ++++++++--------
 2 files changed, 13 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 f31d69f466..287034e98a 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -1129,7 +1129,10 @@ div#facet_sidebar {
     color: [% css_colors.text_alert %];
 }
 #acct_prefs_header { float: left; }
-.search_page_nav_link { cursor: pointer; }
+.search_page_nav_link {
+    cursor: pointer;
+    padding-left: 1em;
+}
 #opac.result.sort { width: 160px; }
 .renew-summary { font-size: [% css_fonts.size_bigger %]; font-style: italic; margin: 0.5ex 0; }
 .failure-text { margin-left: 4em; font-style: italic; color: [% css_colors.text_alert %]; }
@@ -1190,10 +1193,7 @@ a.dash-link:hover { text-decoration: underline !important; }
     padding-bottom: 1ex;
 }
 .cn_browse_item { padding: 2ex; }
-.results-paginator-list {
-    padding-left: 1em;
-    padding-right: 1em;
-}
+.results-paginator-list { padding-left: 1em; }
 .results-paginator-selected { color: [% css_colors.text_alert %]; }
 .inactive-hold { background: [% css_colors.accent_lightest %]; }
 
diff --git a/Open-ILS/src/templates/opac/parts/result/paginate.tt2 b/Open-ILS/src/templates/opac/parts/result/paginate.tt2
index d41822f12d..5673a85717 100644
--- a/Open-ILS/src/templates/opac/parts/result/paginate.tt2
+++ b/Open-ILS/src/templates/opac/parts/result/paginate.tt2
@@ -18,16 +18,16 @@
     <span>
                 <span class='start_end_links_span'>
 
-                    [%  class = 'search_page_nav_link';
+                    [%- class = 'search_page_nav_link';
                         href = '#';
                         IF page > 0;
                             href = mkurl('', {page => page - 1});
-                        ELSE; class = class _ ' invisible'; END;
-                    %]
+                    -%]
                     <a class='[% class %]' href='[% href %]' 
                         title='[% l("Previous page") %]'><span class="nav_arrow_fix">&#9668;</span> [% l('Previous') %] </a>
+                    [%- END;
 
-                    [%- # show links to 4 previous pages, the current page, and 3 more pages.
+                        # show links to 4 previous pages, the current page, and 3 more pages.
                         added = 0;
                         pageitr = page - 5; 
                         IF page_count > 1; # no need to paginate 1 page
@@ -46,16 +46,16 @@
                                 END;
                     -%]
                     </span>
-                    [%- END; -%]
+                    [%- END;
 
-                    [%  class = 'search_page_nav_link';
+                        class = 'search_page_nav_link';
                         href = '#';
                         IF (page + 1) < page_count;
                             href = mkurl('', {page => page + 1});
-                        ELSE; class = class _ ' invisible'; END;
-                    %]
+                    -%]
                     <a class='[% class %]' href='[% href %]' 
                         title='[% l("Next page") %]'> [% l('Next') %] <span class="nav_arrow_fix">&#9658;</span></a>
+                    [%- END; -%]
                 </span>
     </span>
 </div>
-- 
2.11.0