fix: decode the value of location.hash as event callback params

parksben · Jan 4, 2020 · 9a284a0 · 9a284a0
1 parent 9f38d1b
commit 9a284a0
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/index.js b/src/index.js
@@ -186,7 +186,8 @@ export class MarkdownNavbar extends Component {
     return headingList;
   }
 
-  _getCurrentHashValue = () => window.location.hash.replace(/^#/, '');
+  _getCurrentHashValue = () =>
+    decodeURIComponent(window.location.hash.replace(/^#/, ''));
 
   _winScroll = () => {
     const scrollTop =
@@ -209,7 +210,9 @@ export class MarkdownNavbar extends Component {
 
     if (this.props.updateHashAuto) {
       // Hash changing callback
-      this.props.onHashChange(curHeading.dataId, this._getCurrentHashValue());
+      if (curHeading.dataId !== this._getCurrentHashValue()) {
+        this.props.onHashChange(curHeading.dataId, this._getCurrentHashValue());
+      }
 
       this._updateHash(curHeading.dataId);
     }