From 404beb20630cb0897cb57a73eb4f19c62c92ac50 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=ADctor=20Losada=20Hern=C3=A1ndez?= Date: Wed, 2 Sep 2026 13:43:07 +0200 Subject: [PATCH] reducing lines --- .../pages/basePages/listPage/listPage.jsx | 38 +++++++++++-------- 1 file changed, 22 insertions(+), 16 deletions(-) diff --git a/client/homebrew/pages/basePages/listPage/listPage.jsx b/client/homebrew/pages/basePages/listPage/listPage.jsx index 0f5c4a877..0119a40c2 100644 --- a/client/homebrew/pages/basePages/listPage/listPage.jsx +++ b/client/homebrew/pages/basePages/listPage/listPage.jsx @@ -24,14 +24,22 @@ const ListPage = ({ brewCollection = [{ title: '', class: '', brews: [] }], navI const sortTypeRef = useRef(sortType); const sortDirRef = useRef(sortDir); - useEffect(()=>(groupVisibilityRef.current = groupVisibility), [groupVisibility]); - useEffect(()=>(sortTypeRef.current = sortType), [sortType]); - useEffect(()=>(sortDirRef.current = sortDir), [sortDir]); + useEffect(()=>{ + groupVisibilityRef.current = groupVisibility; + }, [groupVisibility]); + + useEffect(()=>{ + sortTypeRef.current = sortType; + }, [sortType]); + + useEffect(()=>{ + sortDirRef.current = sortDir; + }, [sortDir]); useEffect(()=>{ window.onbeforeunload = saveToLocalStorage; - if(typeof window === 'undefined') return; + const newSortType = sortType ?? (localStorage.getItem(USERPAGE_SORT_TYPE) || DEFAULT_SORT_TYPE); const newSortDir = sortDir ?? (localStorage.getItem(USERPAGE_SORT_DIR) || DEFAULT_SORT_DIR); updateUrl(filterString, newSortType, newSortDir); @@ -61,18 +69,16 @@ const ListPage = ({ brewCollection = [{ title: '', class: '', brews: [] }], navI const renderBrews = (brews)=>{ if(!brews || !brews.length) return
No Brews.
; - return _.map(brews, (brew, idx)=>{ - return ( - { - updateUrl(filterString, sortType, sortDir, tag); - }} - /> - ); - }); + return _.map(brews, (brew, idx)=>( + { + updateUrl(filterString, sortType, sortDir, tag); + }} + /> + )); }; const sortBrewOrder = (brew)=>{