From 720e43e9d90c7322ddc2f190893a8bb7781e694b Mon Sep 17 00:00:00 2001
From: Gazook89 <John.W.Jones89@gmail.com>
Date: Mon, 19 Aug 2024 22:43:41 -0500
Subject: [PATCH] update page increment/decrement buttons

putting the `pageInput - 1` modifier in the `scrollToPage` method allows for more understandable button function ('back button' is `- 1` and 'forward button' is `+ 1`).
---
 client/homebrew/brewRenderer/brewRenderer.jsx  |  1 +
 .../homebrew/brewRenderer/toolBar/toolBar.jsx  | 18 ++++++------------
 2 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/client/homebrew/brewRenderer/brewRenderer.jsx b/client/homebrew/brewRenderer/brewRenderer.jsx
index 31e95cc66..2c6910a31 100644
--- a/client/homebrew/brewRenderer/brewRenderer.jsx
+++ b/client/homebrew/brewRenderer/brewRenderer.jsx
@@ -82,6 +82,7 @@ const BrewRenderer = (props)=>{
 	}, []);
 
 	const scrollToPage = (pageNumber) => {
+		pageNumber = pageNumber - 1;
 		const iframe = document.getElementById('BrewRenderer');
         if (iframe && iframe.contentWindow) {
             const brewRenderer =
diff --git a/client/homebrew/brewRenderer/toolBar/toolBar.jsx b/client/homebrew/brewRenderer/toolBar/toolBar.jsx
index 20eb5aee0..3927ee432 100644
--- a/client/homebrew/brewRenderer/toolBar/toolBar.jsx
+++ b/client/homebrew/brewRenderer/toolBar/toolBar.jsx
@@ -64,14 +64,12 @@ const ToolBar = ({ onZoomChange, currentPage, onPageChange, totalPages })=>{
 					<i className='fas fa-magnifying-glass-plus' />
 				</button>
 			</div>
+
 			<div className='tool'>
 				<button
 					className='previousPage'
-					onClick={()=>{
-						console.log(`page is ${state.currentPage}`);
-						onPageChange(state.currentPage - 2);
-					}}
-					disabled={state.currentPage <= 1}
+					onClick={()=>onPageChange(pageInput - 1)}
+					disabled={pageInput <= 1}
 				>
 					<i className='fas fa-arrow-left'></i>
 				</button>
@@ -92,13 +90,9 @@ const ToolBar = ({ onZoomChange, currentPage, onPageChange, totalPages })=>{
 			<div className='tool'>
 				<button
 					className='nextPage'
-					onClick={()=>{
-						console.log(
-							`page is ${state.currentPage} and i move to ${state.currentPage}`
-						);
-						onPageChange(state.currentPage);
-					}}
-					disabled={state.currentPage >= state.totalPages}
+					// onClick={()=>{setPageInput((pageInput)=>parseInt(pageInput) + 1)}}
+					onClick={()=>onPageChange(pageInput + 1)}
+					disabled={pageInput >= totalPages}
 				>
 					<i className='fas fa-arrow-right'></i>
 				</button>