Fix: Correct the navigation to anchor (#17910)

Dedelweiss · Kordyjan · commit bbf95d9f36e8 · 2023-11-17T17:39:03.000+01:00
The browser compatibility of [`location.hash`](https://developer.mozilla.org/en-US/docs/Web/API/Location/hash) seems to be out of date. That's why in this fix, I suggest using [`history.replaceState`](https://developer.mozilla.org/en-US/docs/Web/API/History/replaceState). ## Before: https://github.com/lampepfl/dotty/assets/44496264/a1399b0b-e4e8-4754-b146-661b59a01824 ## After: https://github.com/lampepfl/dotty/assets/44496264/22f38852-0cc5-48eb-a6e7-699f11628401 Fixes: #17182 [Cherry-picked 7694985]
diff --git a/scaladoc/resources/dotty_res/scripts/ux.js b/scaladoc/resources/dotty_res/scripts/ux.js
@@ -285,7 +285,7 @@ document
         getTocListElement(id).parentElement.classList.toggle("active");
       }
       if (lastEntry.isIntersecting) {
-        window.location.hash = "";
+        history.replaceState(history.state, "", window.location.pathname + window.location.search);
         removeAllHighlights();
         const id = getIdOfElement(lastEntry);
 

Original file line number	Diff line number	Diff line change
`@@ -285,7 +285,7 @@ document`
`285`	`285`	`getTocListElement(id).parentElement.classList.toggle("active");`
`286`	`286`	`}`
`287`	`287`	`if (lastEntry.isIntersecting) {`
`288`		`- window.location.hash = "";`
	`288`	`+ history.replaceState(history.state, "", window.location.pathname + window.location.search);`
`289`	`289`	`removeAllHighlights();`
`290`	`290`	`const id = getIdOfElement(lastEntry);`
`291`	`291`