Use more specific selector for name links (#29679) (#29681)

Backport #29679 by @silverwind Followup go-gitea/gitea#29305. As per discussion in go-gitea/gitea#29666 (comment), make this selector only search in the current `.markup` document, as there can be multiples displayed at the same time. @DanielMatiasCarvalho maybe you can review. Co-authored-by: silverwind <[email protected]> (cherry picked from commit 25b0c99a41e9024d439deaa55be7ba87f6cd557f)
DennisRasey · Mar 10, 2024 · ac05e20 · ac05e20
1 parent 03caefb
commit ac05e20
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/web_src/js/markup/anchors.js b/web_src/js/markup/anchors.js
@@ -39,7 +39,7 @@ export function initMarkupAnchors() {
     if (!href.startsWith('#user-content-')) continue;
     const originalId = href.replace(/^#user-content-/, '');
     a.setAttribute('href', `#${encodeURIComponent(originalId)}`);
-    if (document.getElementsByName(originalId).length !== 1) {
+    if (a.closest('.markup').querySelectorAll(`a[name="${originalId}"]`).length !== 1) {
       a.addEventListener('click', (e) => {
         scrollToAnchor(e.currentTarget.getAttribute('href'), false);
       });