From 816ffba74b8bebffc620af50994833d783207a50 Mon Sep 17 00:00:00 2001
From: Keith Carangelo <mail@kcaran.com>
Date: Sat, 29 Aug 2020 11:02:59 -0400
Subject: [PATCH] Added $links_per_page variable to template and display on
 default

---
 application/render/PageBuilder.php |  2 ++
 assets/default/scss/shaarli.scss   |  5 +++++
 tpl/default/linklist.paging.html   | 13 +++++++++----
 3 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/application/render/PageBuilder.php b/application/render/PageBuilder.php
index 7a716673..21703639 100644
--- a/application/render/PageBuilder.php
+++ b/application/render/PageBuilder.php
@@ -149,6 +149,8 @@ class PageBuilder
 
         $this->tpl->assign('formatter', $this->conf->get('formatter', 'default'));
 
+        $this->tpl->assign('links_per_page', $_SESSION['LINKS_PER_PAGE']);
+
         // To be removed with a proper theme configuration.
         $this->tpl->assign('conf', $this->conf);
     }
diff --git a/assets/default/scss/shaarli.scss b/assets/default/scss/shaarli.scss
index 759dff29..3d25d231 100644
--- a/assets/default/scss/shaarli.scss
+++ b/assets/default/scss/shaarli.scss
@@ -618,6 +618,11 @@ body,
     color: $dark-grey;
   }
 
+  a.selected {
+    background: var(--main-color);
+    color: $white;
+  }
+
   input {
     &[type='text'] {
       @extend %linksperpage-button;
diff --git a/tpl/default/linklist.paging.html b/tpl/default/linklist.paging.html
index 7b320eaf..009692b9 100644
--- a/tpl/default/linklist.paging.html
+++ b/tpl/default/linklist.paging.html
@@ -53,11 +53,16 @@
 
     <div class="linksperpage pure-u-1-3">
       <div class="pure-u-0 pure-u-lg-visible">{'Links per page'|t}</div>
-      <a href="{$base_path}/links-per-page?nb=20">20</a>
-      <a href="{$base_path}/links-per-page?nb=50">50</a>
-      <a href="{$base_path}/links-per-page?nb=100">100</a>
+      <a href="{$base_path}/links-per-page?nb=20"
+			{if="$links_per_page == 20"}class="selected"{/if}>20</a>
+      <a href="{$base_path}/links-per-page?nb=50"
+			{if="$links_per_page == 50"}class="selected"{/if}>50</a>
+      <a href="{$base_path}/links-per-page?nb=100"
+			{if="$links_per_page == 100"}class="selected"{/if}>100</a>
       <form method="GET" class="pure-u-0 pure-u-lg-visible" action="{$base_path}/links-per-page">
-        <input type="text" name="nb" placeholder="133">
+        <input type="text" name="nb" placeholder="133"
+			{if="$links_per_page != 20 && $links_per_page != 50 && $links_per_page != 100"}
+				value="{$links_per_page}"{/if}>
       </form>
       <a href="#" class="filter-off fold-all pure-u-0 pure-u-lg-visible" aria-label="{'Fold all'|t}" title="{'Fold all'|t}">
         <i class="fa fa-chevron-up" aria-hidden="true"></i>
-- 
2.41.0