Skip to content

Conversation

@mickael-menu
Copy link
Member

Added

Navigator

  • Added swipe gesture support for navigating in PDF paginated spread mode.

Deprecated

Navigator

  • PDFNavigatorViewController.scalesDocumentToFit is now deprecated and non-functional. The navigator always scales the document to fit the viewport.

Fixed

Navigator

  • Fixed zoom-to-fit scaling in PDF paginated spread mode when offsetFirstPage is enabled.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances the PDF navigator by adding swipe gesture support for navigating in paginated spread mode, deprecating the scalesDocumentToFit property, and fixing zoom-to-fit scaling issues when offsetFirstPage is enabled.

Key Changes

  • Added swipe gesture recognizers (left/right) for page navigation in PDF paginated spread mode
  • Deprecated scalesDocumentToFit property, making the navigator always scale documents to fit the viewport
  • Introduced visiblePagesDidChange notification observer to refresh scale factors when visible pages change in paginated mode

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
Sources/Navigator/PDF/PDFNavigatorViewController.swift Added swipe gesture support, deprecated scalesDocumentToFit property, refactored zoom-to-fit logic with new updateScaleFactors(zoomToFit:) parameter, and added visible pages change observer
CHANGELOG.md Documented the new swipe gesture feature, deprecation of scalesDocumentToFit, and the zoom-to-fit scaling fix

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mickael-menu mickael-menu merged commit fae59aa into develop Dec 2, 2025
11 checks passed
@mickael-menu mickael-menu deleted the fix/pdf-navigator branch December 2, 2025 16:55
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.

2 participants