Skip to content

Conversation

@pratyush3124
Copy link

This commit introduces a refined "Scroll" page view mode that limits vertical scrolling to the current chapter. This addresses the previous issue where continuous scroll modes would either be choppy or scroll across the entire book.

Key changes include:

  • Configured EPUB.js rendition with 'scrolled' flow and removed the explicit 'continuous' manager to enable chapter-limited scrolling.
  • Modified keyboard event handling in Reader.tsx to allow explicit chapter navigation (ArrowLeft/Right) when in scroll mode.
  • Adjusted wheel event handling in Reader.tsx to prevent automatic chapter transitions, ensuring users can scroll freely within the current chapter.

These changes provide a fluid and responsive scrolling experience while maintaining clear chapter boundaries and user control over navigation.

This commit introduces a refined "Scroll" page view mode that limits vertical scrolling to the current chapter. This addresses the previous issue where continuous scroll modes would either be choppy or scroll across the entire book.

Key changes include:
- Configured EPUB.js rendition with 'scrolled' flow and removed the explicit 'continuous' manager to enable chapter-limited scrolling.
- Modified keyboard event handling in [Reader.tsx](cci:7://file:///Users/pratyushkongalla/space/other/flow/apps/reader/src/components/Reader.tsx:0:0-0:0) to allow explicit chapter navigation (ArrowLeft/Right) when in scroll mode.
- Adjusted wheel event handling in [Reader.tsx](cci:7://file:///Users/pratyushkongalla/space/other/flow/apps/reader/src/components/Reader.tsx:0:0-0:0) to prevent automatic chapter transitions, ensuring users can scroll freely within the current chapter.

These changes provide a fluid and responsive scrolling experience while maintaining clear chapter boundaries and user control over navigation.
@vercel
Copy link

vercel bot commented Jun 8, 2025

@pratyush3124 is attempting to deploy a commit to the pacexy Team on Vercel.

A member of the Team first needs to authorize it.

@pratyush3124
Copy link
Author

#168 & #167 are addressed here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant