make unmount effect more explicit
Signed-off-by: Morgan Bentell <mbentell@spotify.com>
This commit is contained in:
@@ -82,7 +82,10 @@ export const useShadowRootSelection = (waitMillis: number = 0) => {
|
||||
[shadowRoot, setSelection, waitMillis],
|
||||
);
|
||||
|
||||
useEffect(() => handleSelectionChange.cancel, [handleSelectionChange.cancel]);
|
||||
useEffect(() => {
|
||||
// cancel debounce on unmount
|
||||
return () => handleSelectionChange.cancel();
|
||||
}, [handleSelectionChange]);
|
||||
|
||||
useEffect(() => {
|
||||
window.document.addEventListener('selectionchange', handleSelectionChange);
|
||||
|
||||
Reference in New Issue
Block a user