From 54b11b1a4c9701a9b59556fa1b9a57374ad9a853 Mon Sep 17 00:00:00 2001 From: Gazook89 Date: Mon, 19 Aug 2024 22:34:32 -0500 Subject: [PATCH] refactor useEffect for updating currentPage state The current page and the current page input share the same value, so the useEffect hook can be simplified. --- client/homebrew/brewRenderer/toolBar/toolBar.jsx | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/client/homebrew/brewRenderer/toolBar/toolBar.jsx b/client/homebrew/brewRenderer/toolBar/toolBar.jsx index 8c2d7464c..87e8a1077 100644 --- a/client/homebrew/brewRenderer/toolBar/toolBar.jsx +++ b/client/homebrew/brewRenderer/toolBar/toolBar.jsx @@ -16,12 +16,8 @@ const ToolBar = ({ onZoomChange, currentPage, onPageChange, totalPages })=>{ }, [zoomLevel]); useEffect(()=>{ - setState((prevState)=>({ - ...prevState, - currentPage : currentPage, - pageNumberInput : currentPage, - })); - }, [currentPage]); + setPageInput(currentPage); + }, [currentPage]) const handleZoomChange = (delta)=>{ const zoomChange = _.clamp(zoomLevel + delta, minZoom, maxZoom);