From 96aa7af4fb3436751e1e0747d177f52471a577fc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=B6ren=20Weber?= <mail@soeren-weber.de>
Date: Sun, 3 Jul 2022 15:40:32 +0200
Subject: [PATCH] history: don't reload page when history gets cleared #299

---
 layouts/partials/menu.html |  2 +-
 static/css/theme.css       |  4 +++
 static/js/theme.js         | 55 +++++++++++++++++++++-----------------
 3 files changed, 36 insertions(+), 25 deletions(-)

diff --git a/layouts/partials/menu.html b/layouts/partials/menu.html
index 9dafd69b5b..9de163a3bb 100644
--- a/layouts/partials/menu.html
+++ b/layouts/partials/menu.html
@@ -102,7 +102,7 @@
               </a>
               <script>variants.markSelectedVariant();</script>
             </li>
-            <li class="footerVisitedLinks{{if $showvisitedlinks}} showVisitedLinks{{end}}"><a class="padding" href="#" data-clear-history-toggle=""><i class="fas fa-history fa-fw"></i> {{ T "Clear-History" }}</a></li>
+            <li class="footerVisitedLinks{{if $showvisitedlinks}} showVisitedLinks{{end}}"><a class="padding" onclick="clearHistory();"><i class="fas fa-history fa-fw"></i> {{ T "Clear-History" }}</a></li>
           </ul>
         </div>
         <div id="footer" class="footerFooter{{if $showfooter}} showFooter{{end}}">
diff --git a/static/css/theme.css b/static/css/theme.css
index 6e6f116114..fb628693e8 100644
--- a/static/css/theme.css
+++ b/static/css/theme.css
@@ -1379,6 +1379,10 @@ rapi-doc {
     display: initial;
 }
 
+.footerVisitedLinks a {
+	cursor: pointer;
+}
+
 @media screen and (max-width: 47.938em) {
     #breadcrumbs .links {
         display: none;
diff --git a/static/js/theme.js b/static/js/theme.js
index caa112f96c..0d609049a3 100644
--- a/static/js/theme.js
+++ b/static/js/theme.js
@@ -648,6 +648,36 @@ function initSwipeHandler(){
     document.querySelectorAll( '#sidebar *' ).forEach( function(e){ e.addEventListener("touchend", handleEndX); }, false);
 }
 
+function clearHistory() {
+    var visitedItem = baseUriFull + 'visited-url/'
+    for( var item in sessionStorage ){
+        if( item.substring( 0, visitedItem.length ) === visitedItem ){
+            sessionStorage.removeItem( item );
+            var url = item.substring( visitedItem.length );
+            // in case we have `relativeURLs=true` we have to strip the
+            // relative path to root
+            url = url.replace( /\.\.\//g, '/' ).replace( /^\/+\//, '/' );
+            jQuery('[data-nav-id="' + url + '"]').removeClass('visited');
+        }
+    }
+}
+
+function initHistory() {
+    var visitedItem = baseUriFull + 'visited-url/'
+    sessionStorage.setItem(visitedItem+jQuery('body').data('url'), 1);
+
+    // loop through the sessionStorage and see if something should be marked as visited
+    for( var item in sessionStorage ){
+        if( item.substring( 0, visitedItem.length ) === visitedItem && sessionStorage.getItem( item ) == 1 ){
+            var url = item.substring( visitedItem.length );
+            // in case we have `relativeURLs=true` we have to strip the
+            // relative path to root
+            url = url.replace( /\.\.\//g, '/' ).replace( /^\/+\//, '/' );
+            jQuery('[data-nav-id="' + url + '"]').addClass('visited');
+        }
+    }
+}
+
 function scrollToActiveMenu() {
     window.setTimeout(function(){
         var e = document.querySelector( '#sidebar ul.topics li.active a' );
@@ -732,16 +762,7 @@ jQuery(function() {
     initCodeClipboard();
     restoreTabSelections();
     initSwipeHandler();
-
-    jQuery('[data-clear-history-toggle]').on('click', function() {
-        for( var item in sessionStorage ){
-          if( item.substring( 0, baseUriFull.length ) === baseUriFull ){
-            sessionStorage.removeItem( item );
-          }
-        }
-        location.reload();
-        return false;
-    });
+    initHistory();
 
     var ajax;
     jQuery('[data-search-input]').on('input', function() {
@@ -798,20 +819,6 @@ jQuery(function() {
 
     $('#topbar a:not(:has(img)):not(.btn)').addClass('highlight');
     $('#body-inner a:not(:has(img)):not(.btn):not(a[rel="footnote"])').addClass('highlight');
-
-    var visitedItem = baseUriFull + 'visited-url/'
-    sessionStorage.setItem(visitedItem+jQuery('body').data('url'), 1);
-
-    // loop through the sessionStorage and see if something should be marked as visited
-    for( var item in sessionStorage ){
-        if( item.substring( 0, visitedItem.length ) === visitedItem && sessionStorage.getItem( item ) == 1 ){
-            var url = item.substring( visitedItem.length );
-            // in case we have `relativeURLs=true` we have to strip the
-            // relative path to root
-            url = url.replace( /\.\.\//g, '/' ).replace( /^\/+\//, '/' );
-            jQuery('[data-nav-id="' + url + '"]').addClass('visited');
-        }
-    }
 });
 
 jQuery.extend({