LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

fix useWindowSize (again)

Browse source
This commit is contained in:
Asad Umar 2022-02-04 23:37:46 +05:00 committed by Thomas Zarebczan
parent e45ef63d41
commit df320a2375
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
ui/effects/use-screensize.js
View file

@ -1,16 +1,20 @@
// Widths are taken from "ui/scss/init/vars.scss" // Widths are taken from "ui/scss/init/vars.scss"
import React from 'react'; import React, { useRef } from 'react';
export function useWindowSize(fn) { export function useWindowSize(fn) {
const isWindowClient = typeof window === 'object'; const isWindowClient = typeof window === 'object';
const initialState = fn ? fn(window.innerWidth) : window.innerWidth; const initialState = fn ? fn(window.innerWidth) : window.innerWidth;
const [windowSize, setWindowSize] = React.useState(isWindowClient ? initialState : undefined); const [windowSize, setWindowSize] = React.useState(isWindowClient ? initialState : undefined);
const prev = useRef();
React.useEffect(() => { React.useEffect(() => {
function setSize() { function setSize() {
if (fn) { if (fn) {
const curr = fn(window.innerWidth); const curr = fn(window.innerWidth);
if (windowSize !== curr) setWindowSize(curr); if (prev !== curr) {
setWindowSize(curr);
prev.current = curr;
}
} else setWindowSize(window.innerWidth); } else setWindowSize(window.innerWidth);
} }
@ -21,7 +25,7 @@ export function useWindowSize(fn) {
return () => window.removeEventListener('resize', setSize); return () => window.removeEventListener('resize', setSize);
} }
}, [isWindowClient, setWindowSize]); }, [fn, isWindowClient, setWindowSize, windowSize]);
return windowSize; return windowSize;
} }