f.weber/homebrewery
0
0
Fork 0
mirror of https://github.com/naturalcrit/homebrewery.git synced 2025-12-26 03:02:40 +00:00
Code Wiki Activity

Move formatVisiblePages

Browse Source 
After simplifying, this has become a single-line function used in only one place. Can just be placed directly in the one place it is used.
This commit is contained in:
Trevor Buckner
2024-12-24 00:24:52 -05:00
parent 0632d78f71
commit 8159c408c8
1 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
client/homebrew/brewRenderer/toolBar/toolBar.jsx
View File

@@ -15,7 +15,9 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa
const [toolsVisible, setToolsVisible] = useState(true);
useEffect(()=>{
setPageNum(formatVisiblePages(visiblePages));
// format multiple visible pages as a range (e.g. "150-153")
const pageRange = visiblePages.length === 1 ? `${visiblePages[0]}` : `${visiblePages[0]} - ${visiblePages.at(-1)}`;
setPageNum(pageRange);
}, [visiblePages]);
const handleZoomButton = (zoom)=>{
@@ -72,11 +74,6 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa
return deltaZoom;
};
// format the visible pages into a range (e.g. "150-153")
const formatVisiblePages = (pages)=>{
return pages.length === 1 ? `${pages[0]}` : `${pages[0]} - ${pages.at(-1)}`;
};
return (
<div id='preview-toolbar' className={`toolBar ${toolsVisible ? 'visible' : 'hidden'}`} role='toolbar'>
<button className='toggleButton' title={`${toolsVisible ? 'Hide' : 'Show'} Preview Toolbar`} onClick={()=>{setToolsVisible(!toolsVisible);}}><i className='fas fa-glasses' /></button>