From 499c640a112b0b3b05dc61f90832b0269ccb055b Mon Sep 17 00:00:00 2001
From: Gazook89 <John.W.Jones89@gmail.com>
Date: Sun, 18 Aug 2024 18:50:36 -0500
Subject: [PATCH] Move the Toolbar component out of the iframe

The website UI should stay out of the iframe if possible.  Otherwise, users can style the UI on their brew which may or may not be a pain, and it's simple enough to avoid it.
---
 client/homebrew/brewRenderer/brewRenderer.jsx | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/client/homebrew/brewRenderer/brewRenderer.jsx b/client/homebrew/brewRenderer/brewRenderer.jsx
index 4451d31b2..ce9d69418 100644
--- a/client/homebrew/brewRenderer/brewRenderer.jsx
+++ b/client/homebrew/brewRenderer/brewRenderer.jsx
@@ -231,10 +231,10 @@ const BrewRenderer = (props)=>{
 		}));
 	};
 
-	const handlePageChange = (pageNumber) => {
-        // Scroll to the desired page
-        scrollToPage(pageNumber);
-    };
+	// const handlePageChange = (pageNumber) => {
+    //     // Scroll to the desired page
+    //     scrollToPage(pageNumber);
+    // };
 
 	return (
 		<>
@@ -253,13 +253,15 @@ const BrewRenderer = (props)=>{
 				<NotificationPopup />
 			</div>
 
+			<ToolBar onZoomChange={handleZoom} currentPage={state.currentPageNumber} onPageChange={scrollToPage} totalPages={rawPages.length}/>
+
+
 			{/*render in iFrame so broken code doesn't crash the site.*/}
 			<Frame id='BrewRenderer' initialContent={INITIAL_CONTENT}
 				style={{ width: '100%', height: '100%', visibility: state.visibility }}
 				contentDidMount={frameDidMount}
 				onClick={()=>{emitClick();}}
 			>
-				<ToolBar onZoomChange={handleZoom} currentPage={state.currentPageNumber} onPageChange={handlePageChange} totalPages={rawPages.length}/>
 				<div className={'brewRenderer'}
 					onScroll={handleScroll}
 					onKeyDown={handleControlKeys}