web/timeline: fix clicking spoilers and summaries on mobile

tulir · Jan 23, 2025 · 15238b6 · 15238b6
1 parent ce72841
commit 15238b6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/web/src/ui/timeline/TimelineEvent.tsx b/web/src/ui/timeline/TimelineEvent.tsx
@@ -106,6 +106,8 @@ const TimelineEvent = ({ evt, prevEvt, disableMenu, smallReplies, isFocused }: T
 		if (
 			targetElem.tagName === "A"
 			|| targetElem.tagName === "IMG"
+			|| targetElem.tagName === "VIDEO"
+			|| targetElem.tagName === "SUMMARY"
 		) {
 			return
 		}

diff --git a/web/src/ui/timeline/content/TextMessageBody.tsx b/web/src/ui/timeline/content/TextMessageBody.tsx
@@ -58,9 +58,11 @@ const onClickHTML = (evt: React.MouseEvent<HTMLDivElement>) => {
 	} else if (targetElem.closest?.("span.hicli-spoiler")?.classList.toggle("spoiler-revealed")) {
 		// When unspoilering, don't trigger links and other clickables inside the spoiler
 		evt.preventDefault()
+		evt.stopPropagation()
 	} else if (isAnchorElement(targetElem) && targetElem.href.startsWith("matrix:")) {
 		onClickMatrixURI(targetElem.href)
 		evt.preventDefault()
+		evt.stopPropagation()
 	}
 }