From 8be3154865ec6dfff5e2fabba85a0787eeb03949 Mon Sep 17 00:00:00 2001 From: Gazook89 Date: Sat, 17 Aug 2024 20:48:30 -0500 Subject: [PATCH] rename functions/attributes updateZoom() => handleZoom updateZoom prop => onZoomChange More clarity of prop versus handler function. --- client/homebrew/brewRenderer/brewRenderer.jsx | 4 ++-- client/homebrew/brewRenderer/toolBar/toolBar.jsx | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/client/homebrew/brewRenderer/brewRenderer.jsx b/client/homebrew/brewRenderer/brewRenderer.jsx index 845ae612d..4451d31b2 100644 --- a/client/homebrew/brewRenderer/brewRenderer.jsx +++ b/client/homebrew/brewRenderer/brewRenderer.jsx @@ -224,7 +224,7 @@ const BrewRenderer = (props)=>{ }; //Toolbar settings: - const updateZoom = (newZoom) => { + const handleZoom = (newZoom) => { setState((prevState)=>({ ...prevState, zoom : newZoom @@ -259,7 +259,7 @@ const BrewRenderer = (props)=>{ contentDidMount={frameDidMount} onClick={()=>{emitClick();}} > - +
{ +const ToolBar = ({ onZoomChange, currentPage, onPageChange, totalPages })=>{ const [state, setState] = useState({ currentPage : currentPage, totalPages : totalPages, @@ -16,7 +16,7 @@ const ToolBar = ({ updateZoom, currentPage, onPageChange, totalPages })=>{ }); useEffect(()=>{ - updateZoom(state.zoomLevel); + onZoomChange(state.zoomLevel); }, [state.zoomLevel]); useEffect(()=>{ @@ -87,7 +87,7 @@ const ToolBar = ({ updateZoom, currentPage, onPageChange, totalPages })=>{ zoomLevel : newZoomLevel, zoomInput : newZoomLevel, })); - updateZoom(newZoomLevel); + onZoomChange(newZoomLevel); } }} />