ChoChoX/suwayomi-material-you-webui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix TypeError when reaching scroll end in continuous pager

Browse Source 
When the end of the scrollable element is reached in the continuous reader, the last image is not visible anymore.
Thus, the "firstVisibleImageIndex" is invalid and gets incorrectly used for the "isEndReached" case which results in an TypeError when trying to use the actual page via the index
This commit is contained in:
schroda
2024-12-31 16:54:21 +01:00
parent 8c147b797a
commit 886cc469bc
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/modules/reader/services/ReaderControls.ts
View File

@@ -465,15 +465,17 @@ export class ReaderControls {
const lastPage = imageRefs.current?.[imageRefs.current.length - 1]; const lastPage = imageRefs.current?.[imageRefs.current.length - 1];
const isEndReached = lastPage && isEndOfPageInViewport(lastPage, type, readingDirection); const isEndReached = lastPage && isEndOfPageInViewport(lastPage, type, readingDirection);
if (firstVisibleImageIndex === -1) {
return;
}
// handle cases where the last page is too small to ever be the "firstVisibleImageIndex" // handle cases where the last page is too small to ever be the "firstVisibleImageIndex"
if (isEndReached) { if (isEndReached) {
updateCurrentPageIndex(firstVisibleImageIndex, false, true); updateCurrentPageIndex(firstVisibleImageIndex, false, true);
return; return;
} }
if (firstVisibleImageIndex !== -1) { updateCurrentPageIndex(firstVisibleImageIndex, firstVisibleImageIndex !== lastPageIndex);
updateCurrentPageIndex(firstVisibleImageIndex, firstVisibleImageIndex !== lastPageIndex);
}
} }
static useHandleClick( static useHandleClick(