From cdc2ffeff4f87d5ede1fce33577697a024d9c219 Mon Sep 17 00:00:00 2001
From: Gazook89 <John.W.Jones89@gmail.com>
Date: Tue, 20 Aug 2024 20:18:54 -0500
Subject: [PATCH] remove `if` statement that shouldn't occur at all

Removing a redundant if/else that shouldn't be possible now that the currentPage is clamped between 1 and totalPages.
---
 client/homebrew/brewRenderer/toolBar/toolBar.jsx | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/client/homebrew/brewRenderer/toolBar/toolBar.jsx b/client/homebrew/brewRenderer/toolBar/toolBar.jsx
index e20f0820f..2a4bd3b52 100644
--- a/client/homebrew/brewRenderer/toolBar/toolBar.jsx
+++ b/client/homebrew/brewRenderer/toolBar/toolBar.jsx
@@ -36,12 +36,7 @@ const ToolBar = ({ onZoomChange, currentPage, onPageChange, totalPages })=>{
 			const brewRenderer = iframe.contentWindow.document.querySelector('.brewRenderer');
 			if(brewRenderer) {
 				const pages = brewRenderer.querySelectorAll('.page');
-
-				if(pageNumber + 1 > pages.length || pageNumber < 0) {
-					console.log('page not found');
-				} else {
-					pages[pageNumber].scrollIntoView({ block: 'start' });
-				}
+				pages[pageNumber]?.scrollIntoView({ block: 'start' });
 			}
 		}
 	};