From e9e49e39fb1fc0e084f8d45cf3e57696b11ee486 Mon Sep 17 00:00:00 2001 From: Gazook89 Date: Sat, 17 Aug 2024 13:59:58 -0500 Subject: [PATCH] Set preview zoom to update with onChange Commented out the existing onChange handler, switched the onMouseUp that actually applies the zoom to onChange. --- client/homebrew/brewRenderer/toolBar/toolBar.jsx | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/client/homebrew/brewRenderer/toolBar/toolBar.jsx b/client/homebrew/brewRenderer/toolBar/toolBar.jsx index 37cfcb6d7..cad878647 100644 --- a/client/homebrew/brewRenderer/toolBar/toolBar.jsx +++ b/client/homebrew/brewRenderer/toolBar/toolBar.jsx @@ -78,8 +78,8 @@ const ToolBar = ({ updateZoom, currentPage, onPageChange, totalPages })=>{ max={maxZoom} step={zoomStep} value={state.zoomInput} - onChange={(e)=>handleInputChange(e.target.value, 'zoom')} - onMouseUp={(e)=>{ + // onChange={(e)=>handleInputChange(e.target.value, 'zoom')} + onChange={(e)=>{ const newZoomLevel = parseInt(e.target.value, 10); if(newZoomLevel !== state.zoomLevel) { setState((prevState)=>({ @@ -92,15 +92,7 @@ const ToolBar = ({ updateZoom, currentPage, onPageChange, totalPages })=>{ }} /> - {Array.from( - { - length : - Math.floor((maxZoom - minZoom) / zoomStep) + 1, - }, - (_, i)=>minZoom + i * zoomStep - ).map((option)=>( -