From aa951ff96cdcacc3f2845ba641f990799913929f Mon Sep 17 00:00:00 2001
From: Trevor Buckner <calculuschild@gmail.com>
Date: Mon, 9 Dec 2024 17:04:16 -0500
Subject: [PATCH] Small cleanups

---
 client/homebrew/brewRenderer/brewRenderer.jsx    |  8 +++-----
 client/homebrew/brewRenderer/toolBar/toolBar.jsx | 13 +++++--------
 2 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/client/homebrew/brewRenderer/brewRenderer.jsx b/client/homebrew/brewRenderer/brewRenderer.jsx
index 879af2a9a..7792742cd 100644
--- a/client/homebrew/brewRenderer/brewRenderer.jsx
+++ b/client/homebrew/brewRenderer/brewRenderer.jsx
@@ -32,10 +32,8 @@ const INITIAL_CONTENT = dedent`
 //v=====----------------------< Brew Page Component >---------------------=====v//
 const BrewPage = (props)=>{
 	props = {
-		contents           : '',
-		index              : 0,
-		onVisibilityChange : ()=>{},
-		onCenterPageChange : ()=>{},
+		contents : '',
+		index    : 0,
 		...props
 	};
 	const pageRef = useRef(null);
@@ -77,7 +75,7 @@ const BrewPage = (props)=>{
 			visibleObserver.disconnect();
 			centerObserver.disconnect();
 		};
-	}, [props.index, props.onVisibilityChange, props.onCenterPageChange]);
+	}, []);
 
 	return <div className={props.className} id={`p${props.index + 1}`} data-index={props.index} ref={pageRef} style={props.style}>
 	         <div className='columnWrapper' dangerouslySetInnerHTML={{ __html: cleanText }} />
diff --git a/client/homebrew/brewRenderer/toolBar/toolBar.jsx b/client/homebrew/brewRenderer/toolBar/toolBar.jsx
index c2486c095..9257660d1 100644
--- a/client/homebrew/brewRenderer/toolBar/toolBar.jsx
+++ b/client/homebrew/brewRenderer/toolBar/toolBar.jsx
@@ -25,7 +25,6 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa
 	};
 
 	const handleOptionChange = (optionKey, newValue)=>{
-		//setDisplayOptions(prevOptions => ({ ...prevOptions, [optionKey]: newValue }));
 		onDisplayOptionsChange({ ...displayOptions, [optionKey]: newValue });
 	};
 
@@ -61,12 +60,10 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa
 		} else if(mode == 'fit'){
 			let minDimRatio;
 			// find the page with the largest single dim (height or width) so that zoom can be adapted to fit it.
-			if(displayOptions.spread === 'facing'){
+			if(displayOptions.spread === 'facing')
 				minDimRatio = [...pages].reduce((minRatio, page)=>Math.min(minRatio, iframeWidth / page.offsetWidth / 2), Infinity);    // if 'facing' spread, fit two pages in view
-			} else {
+			else 
 				minDimRatio = [...pages].reduce((minRatio, page)=>Math.min(minRatio, iframeWidth / page.offsetWidth, iframeHeight / page.offsetHeight), Infinity);
-			}
-			console.log(minDimRatio);
 
 			desiredZoom = minDimRatio * 100;
 		}
@@ -86,9 +83,8 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa
 		let start = sortedPages[0];
 
 		for (let i = 1; i <= sortedPages.length; i++) {
-			// If the current page is not consecutive or it's the end of the list
+			// If the current page is the end of the list or not consecutive
 			if(i === sortedPages.length || sortedPages[i] !== sortedPages[i - 1] + 1) {
-				// Push the range to the list
 				ranges.push(
 					start === sortedPages[i - 1] ? `${start}` : `${start} - ${sortedPages[i - 1]}`
 				);
@@ -233,7 +229,7 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa
 						title='Current page(s) in view'
 						inputMode='numeric'
 						pattern='[0-9]'
-						value={`${pageNum}`}
+						value={pageNum}
 						onClick={(e)=>e.target.select()}
 						onChange={(e)=>handlePageInput(e.target.value)}
 						onBlur={()=>scrollToPage(pageNum)}
@@ -257,6 +253,7 @@ const ToolBar = ({ displayOptions, onDisplayOptionsChange, visiblePages, totalPa
 						const start = _.max(visiblePages);
 						scrollToPage(start);
 						if(start === _.max(visiblePages)){
+							console.log("oh no")
 							scrollToPage(start + 1);
 						};
 					}}