n/nyxmuks
1
0
Fork 0
forked from Mirrors/gomuks
Code Pull requests Activity

web/contentvisibility: work around another chrome bug

Browse source
This commit is contained in:
Tulir Asokan 2024-11-02 00:37:33 +02:00
parent 2a89f10589
commit 249dffaa2d
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
web/src/util/contentvisibility.ts
View file

@ -15,7 +15,7 @@
// along with this program. If not, see <https://www.gnu.org/licenses/>. // along with this program. If not, see <https://www.gnu.org/licenses/>.
import { RefObject, useLayoutEffect, useRef, useState } from "react" import { RefObject, useLayoutEffect, useRef, useState } from "react"
export default function useContentVisibilit<T extends HTMLElement>( export default function useContentVisibility<T extends HTMLElement>(
allowRevert = false, allowRevert = false,
): [boolean, RefObject<T | null>] { ): [boolean, RefObject<T | null>] {
const ref = useRef<T>(null) const ref = useRef<T>(null)
@ -25,13 +25,17 @@ export default function useContentVisibilit<T extends HTMLElement>(
if (!element) { if (!element) {
return return
} }
const listener = (evt: unknown) => { const listener = ((evt: ContentVisibilityAutoStateChangeEvent) => {
if (evt.target !== evt.currentTarget) {
// Workaround for chrome bug https://issues.chromium.org/issues/365168180
return
}
if (!(evt as ContentVisibilityAutoStateChangeEvent).skipped) { if (!(evt as ContentVisibilityAutoStateChangeEvent).skipped) {
setVisible(true) setVisible(true)
} else if (allowRevert) { } else if (allowRevert) {
setVisible(false) setVisible(false)
} }
} }) as (evt: unknown) => void
element.addEventListener("contentvisibilityautostatechange", listener) element.addEventListener("contentvisibilityautostatechange", listener)
return () => element.removeEventListener("contentvisibilityautostatechange", listener) return () => element.removeEventListener("contentvisibilityautostatechange", listener)
}, [allowRevert]) }, [allowRevert])