fix: _setActiveLink() doesn't work if is # in url

Author: mrholek

js/src/sidebar.js

@@ -441,21 +441,31 @@ class Sidebar {
   _setActiveLink() {
     // eslint-disable-next-line unicorn/prefer-spread
     Array.from(this._element.querySelectorAll(SELECTOR_NAV_LINK)).forEach(element => {
-      let currentUrl
+      let currentUrl = String(window.location)
 
-      const urlHasParams = /\\?.*=/
-      const urlHasQueryString = /\\?./
+      const urlHasParams = /\?.*=/
+      const urlHasQueryString = /\?./
       const urlHasHash = /#./
 
-      if (urlHasParams.test(String(window.location)) || urlHasQueryString.test(String(window.location))) {
-        currentUrl = String(window.location).split('?')[0]
-      } else if (urlHasHash.test(String(window.location))) {
-        currentUrl = String(window.location).split('#')[0]
-      } else {
-        currentUrl = String(window.location)
+      console.log(urlHasHash.test(String(window.location)))
+
+      if (urlHasParams.test(currentUrl) || urlHasQueryString.test(currentUrl)) {
+        currentUrl = currentUrl.split('?')[0]
       }
 
-      if (currentUrl.slice(- 1) === '#') {
+      if (urlHasHash.test(currentUrl)) {
+        currentUrl = currentUrl.split('#')[0]
+      }
+
+      // if (urlHasParams.test(String(window.location)) || urlHasQueryString.test(String(window.location))) {
+      //   currentUrl = String(window.location).split('?')[0]
+      // } else if (urlHasHash.test(String(window.location))) {
+      //   currentUrl = String(window.location).split('#')[0]
+      // } else {
+      //   currentUrl = String(window.location)
+      // }
+
+      if (currentUrl.slice(-1) === '#') {
         currentUrl = currentUrl.slice(0, -1)
       }