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

Add handlePageChange() to control input

Browse Source 
Reintroducing handlePageChange() method to handle the page input separately from the scroll function.  It tests the regex pattern on the string, parses to integer, and sets pageNum state.

scrollToPage also now sets the pageNum state after performing the scroll so the input box matches the new current page.
This commit is contained in:
Gazook89
2024-08-24 09:05:43 -05:00
parent 370c480670
commit 3acf90dfdb
1 changed files with 14 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -24,16 +24,27 @@ const ToolBar = ({ onZoomChange, currentPage, onPageChange, totalPages })=>{
setZoomLevel(newZoomLevel);
};
const handlePageChange = (page)=>{
const regex = /[0-9]/;
if(regex.test(page)){
const num = parseInt(page); // input type is 'text', so `page` comes in as a string, not number.
setPageNum(num)
} else {
return;
}
};
const scrollToPage = (pageNumber)=>{
pageNumber = _.clamp(pageNumber - 1, 0, totalPages - 1);
pageNumber = _.clamp(pageNumber, 1, totalPages);
const iframe = document.getElementById('BrewRenderer');
if(iframe && iframe.contentWindow) {
const brewRenderer = iframe.contentWindow.document.querySelector('.brewRenderer');
if(brewRenderer) {
const pages = brewRenderer.querySelectorAll('.page');
pages[pageNumber]?.scrollIntoView({ block: 'start' });
pages[pageNumber - 1]?.scrollIntoView({ block: 'start' });
}
}
setPageNum(pageNumber);
};
return (
@@ -92,7 +103,7 @@ const ToolBar = ({ onZoomChange, currentPage, onPageChange, totalPages })=>{
inputMode='numeric'
pattern='[0-9]'
value={pageNum}
onChange={(e)=>{setPageNum(parseInt(e.target.value));}}
onChange={(e)=>{handlePageChange(e.target.value);}}
onBlur={()=>scrollToPage(pageNum)}
onKeyDown={(e)=>{e.key == 'Enter' ? scrollToPage(pageNum) : null;}}
/>