Skip to content

fix(ios): hide vertical pager scroll edge effects - #1149

Merged
troZee merged 1 commit into
callstack:masterfrom
ArekChr:fix/ios-vertical-pager-safe-area
Oct 1, 2026
Merged

troZee merged 1 commit into
callstack:masterfrom
ArekChr:fix/ios-vertical-pager-safe-area

Conversation

@ArekChr

@ArekChr ArekChr commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

On iOS 26, the native scroll edge effects of a vertical pager can appear sideways over the page content. With v9.0.6, the layout and insets are correct in my app, but this overlay remains visible.

This PR hides the four collection-view edge effects for vertical pagers on iOS 26 and later. It does not change the layout, the public API, or Android behavior. The earlier layout and safe-area changes were removed after rebasing onto v9.0.6; #1150 already covers the root GeometryReader adjustment.

Verification

  • Reproduced the visible edge effect in my app with unpatched v9.0.6.
  • Applied the narrowed change to v9.0.6 with patch-package.
  • Built the react-native-pager-view iOS simulator target with Xcode 26.3.
  • Visual verification of this narrowed patch on an iOS 26 device or simulator is still pending.

Screenshots

Reported application screenshots of the issue:

Reported pager issue: content shifted left and covered by a sideways gradient Second application screenshot supplied with the pager issue report

@troZee

troZee commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

@ArekChr

ArekChr commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

@troZee The insets are fixed now, but I can still see the native edge effects overlaying the page content. So the layout itself is correct, but the edge effect is still visible and covers part of the page.

@ArekChr
ArekChr force-pushed the fix/ios-vertical-pager-safe-area branch from b40b95a to 4838bc8 Compare October 1, 2026 07:09
@ArekChr ArekChr changed the title fix(ios): correct vertical pager layout and scroll edge effects fix(ios): hide vertical pager scroll edge effects Oct 1, 2026
@ArekChr

ArekChr commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

@troZee I’ve rebased this on v9.0.6 and resolved the conflict with #1150. Since the insets are fixed now, I removed the layout changes and kept only the edge effect fix.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The iOS 26 visual behavior still requires validation on a device or simulator.

Review effort: Balanced
Findings: None

What changed in this PR

Hides rotated iOS 26 scroll-edge effects that obscure vertical pager content.

Changes:

  • Hides all collection-view edge effects for vertical pagers on iOS 26+.
  • Leaves horizontal pagers and earlier iOS versions unchanged.
File Description
ios/​PagerView.swift Disables edge effects for rotated vertical pagers.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@troZee
troZee merged commit a6db3b8 into callstack:master Oct 1, 2026
2 checks passed
lvlrSajjad added a commit to lvlrSajjad/react-native-pager-view that referenced this pull request Oct 1, 2026
Resolve the conflict in ios/PagerView.swift by keeping the iOS 26 edge
effect hiding for the vertical pager (callstack#1149) ahead of the once-per-collection-view
delegate install.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

3 participants