ekzyis/stacker.news
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

also untrack descendants of an outlined new comment; micro-optimize classList logic (#2463)

Browse Source
This commit is contained in:
soxa 2025-08-29 20:26:46 +02:00 committed by GitHub
parent 6f67aaaef9
commit 4ff3dfa412
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 11 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
components/comment.js
View File

@ -117,15 +117,18 @@ export default function Comment ({
const unsetOutline = () => {
if (!ref.current) return
const hasOutline = ref.current.classList.contains('outline-new-comment') || ref.current.classList.contains('outline-new-injected-comment')
const hasOutlineUnset = ref.current.classList.contains('outline-new-comment-unset')
// don't try to unset the outline if the comment is not outlined or we already unset the outline
if (hasOutline && !hasOutlineUnset) {
ref.current.classList.add('outline-new-comment-unset')
// untrack the new comment
navigator?.untrackNewComment(ref)
}
const classes = ref.current.classList
const hasOutline = classes.contains('outline-new-comment')
const hasInjectedOutline = classes.contains('outline-new-injected-comment')
const hasOutlineUnset = classes.contains('outline-new-comment-unset')
// don't try to untrack and unset the outline if the comment is not outlined or we already unset the outline
if (!(hasInjectedOutline || hasOutline) || hasOutlineUnset) return
classes.add('outline-new-comment-unset')
// untrack new comment and its descendants if it's not a live comment
navigator?.untrackNewComment(ref, { includeDescendants: hasOutline })
}
useEffect(() => {