fix: the exception in Safari due to too many calls to the history.rep…

…lacestate method
parksben · Mar 30, 2020 · 2834372 · 2834372
1 parent 3fd202f
commit 2834372
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 12 deletions.
diff --git a/README.md b/README.md
@@ -97,10 +97,3 @@ ReactDOM.render(<App />, document.getElementById('root'));
 ## 🍺 License
 
 [MIT license](./LICENSE)
-
-## ☕️ Donate me a coffee
-
-<div style="overflow: hidden;">
-  <img style="display: inline-block;" width="auto" height="240" src="https://github.com/parksben/markdown-navbar/blob/master/assets/wechat.jpg?raw=true" alt="wechat" />
-  <img style="display: inline-block;" width="auto" height="240" src="https://github.com/parksben/markdown-navbar/blob/master/assets/alipay.jpg?raw=true" alt="alipay" />
-</div>
diff --git a/assets/alipay.jpg b/assets/alipay.jpg
diff --git a/assets/wechat.jpg b/assets/wechat.jpg
diff --git a/src/index.js b/src/index.js
@@ -245,11 +245,17 @@ export class MarkdownNavbar extends Component {
   };
 
   updateHash(value) {
-    window.history.replaceState(
-      {},
-      '',
-      `${window.location.pathname}${window.location.search}#${value}`
-    );
+    if (this.updateHashTimeout) {
+      clearTimeout(this.updateHashTimeout);
+    }
+
+    this.updateHashTimeout = setTimeout(() => {
+      window.history.replaceState(
+        {},
+        '',
+        `${window.location.pathname}${window.location.search}#${value}`
+      );
+    }, 0);
   }
 
   render() {