fixed expanding comment threads breaking if their titles contain clickable elements

Oaphi · Oaphi · commit 6a65145d53a3 · 2026-04-02T06:32:29.000+03:00
diff --git a/app/assets/javascripts/comments.js b/app/assets/javascripts/comments.js
@@ -41,14 +41,14 @@ $(() => {
     window.hljs && hljs.highlightAll();
   }
 
-  $(document).on('click', '.post--comments-thread.is-inline a', async (ev) => {
+  $(document).on('click', '.js--comment-link', async (ev) => {
     if (ev.ctrlKey || ev.metaKey) {
       return;
     }
 
     ev.preventDefault();
 
-    const $tgt = $(ev.target);
+    const $tgt = $(ev.currentTarget);
     const $threadId = $tgt.data('thread');
     const wrapper = getCommentThreadWrapper($tgt);
 

Original file line number	Diff line number	Diff line change
`@@ -41,14 +41,14 @@ $(() => {`
`41`	`41`	`window.hljs && hljs.highlightAll();`
`42`	`42`	`}`
`43`	`43`
`44`		`- $(document).on('click', '.post--comments-thread.is-inline a', async (ev) => {`
	`44`	`+ $(document).on('click', '.js--comment-link', async (ev) => {`
`45`	`45`	`if (ev.ctrlKey \|\| ev.metaKey) {`
`46`	`46`	`return;`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`ev.preventDefault();`
`50`	`50`
`51`		`- const $tgt = $(ev.target);`
	`51`	`+ const $tgt = $(ev.currentTarget);`
`52`	`52`	`const $threadId = $tgt.data('thread');`
`53`	`53`	`const wrapper = getCommentThreadWrapper($tgt);`
`54`	`54`