From c2cf695c1721e41277f9883f1f80f298cb86a938 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=ADctor=20Losada=20Hern=C3=A1ndez?= Date: Mon, 2 Jun 2025 13:50:51 +0200 Subject: [PATCH 1/3] add defaults --- client/homebrew/brewRenderer/brewRenderer.jsx | 4 +++- client/homebrew/brewRenderer/toolBar/toolBar.jsx | 8 ++++---- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/client/homebrew/brewRenderer/brewRenderer.jsx b/client/homebrew/brewRenderer/brewRenderer.jsx index c391d8c43..114fe7ed6 100644 --- a/client/homebrew/brewRenderer/brewRenderer.jsx +++ b/client/homebrew/brewRenderer/brewRenderer.jsx @@ -113,7 +113,9 @@ const BrewRenderer = (props)=>{ zoomLevel : 100, spread : 'single', startOnRight : true, - pageShadows : true + pageShadows : true, + rowGap : 5, + columnGap : 10, }); //useEffect to store or gather toolbar state from storage diff --git a/client/homebrew/brewRenderer/toolBar/toolBar.jsx b/client/homebrew/brewRenderer/toolBar/toolBar.jsx index 4f3e356a7..81af0d56f 100644 --- a/client/homebrew/brewRenderer/toolBar/toolBar.jsx +++ b/client/homebrew/brewRenderer/toolBar/toolBar.jsx @@ -21,8 +21,8 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa }, [visiblePages]); useEffect(()=>{ - const visibility = localStorage.getItem('hb_toolbarVisibility') === 'true'; - setToolsVisible(visibility); + const visibility = localStorage.getItem('hb_toolbarVisibility'); + if(visibility) setToolsVisible(visibility); }, []); const handleZoomButton = (zoom)=>{ @@ -68,7 +68,7 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa } else if(mode == 'fit'){ // find the page with the largest single dim (height or width) so that zoom can be adapted to fit it. let minDimRatio; - if(displayOptions.spread === 'active') + if(displayOptions.spread === 'single') minDimRatio = [...pages].reduce( (minRatio, page)=>Math.min(minRatio, iframeWidth / page.offsetWidth, @@ -165,7 +165,7 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa id='single-spread' className='tool' title='Single Page' - onClick={()=>{handleOptionChange('spread', 'active');}} + onClick={()=>{handleOptionChange('spread', 'single');}} aria-checked={displayOptions.spread === 'single'} >