From 5077fda3f602700a46c0b012d334a9946a8c59d9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=ADctor=20Losada=20Hern=C3=A1ndez?= Date: Thu, 17 Oct 2024 22:58:14 +0200 Subject: [PATCH] move stuff around for minimal changes --- client/homebrew/brewRenderer/brewRenderer.jsx | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/client/homebrew/brewRenderer/brewRenderer.jsx b/client/homebrew/brewRenderer/brewRenderer.jsx index a2b3cf828..09d77946d 100644 --- a/client/homebrew/brewRenderer/brewRenderer.jsx +++ b/client/homebrew/brewRenderer/brewRenderer.jsx @@ -115,6 +115,14 @@ const BrewRenderer = (props)=>{ }; }, []); + const updateCurrentPage = useCallback(_.throttle((e)=>{ + const { scrollTop, clientHeight, scrollHeight } = e.target; + const totalScrollableHeight = scrollHeight - clientHeight; + const currentPageNumber = Math.max(Math.ceil((scrollTop / totalScrollableHeight) * rawPages.length), 1); + + props.onPageChange(currentPageNumber); + }, 200), []); + const frameDidMount = ()=>{ //This triggers when iFrame finishes internal "componentDidMount" setTimeout(()=>{ //We still see a flicker where the style isn't applied yet, so wait 100ms before showing iFrame renderPages(); //Make sure page is renderable before showing @@ -126,14 +134,6 @@ const BrewRenderer = (props)=>{ }, 100); }; - const updateCurrentPage = useCallback(_.throttle((e)=>{ - const { scrollTop, clientHeight, scrollHeight } = e.target; - const totalScrollableHeight = scrollHeight - clientHeight; - const currentPageNumber = Math.max(Math.ceil((scrollTop / totalScrollableHeight) * rawPages.length), 1); - - props.onPageChange(currentPageNumber); - }, 200), []); - const isInView = (index)=>{ if(!state.isMounted) return false;