react / react/react-native

[iOS][Fabric] ScrollView.scrollToEnd ignores adjustedContentInset

Đang mở Phù hợp với người mới
#57,522 0 bình luận 1 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Ngôn ngữ chính
C++
Star
127k
Fork
25.3k
Merge trung bình
1 ngày 23 giờ
Pull request đã merge (30 ngày)
4

Mô tả

Description

On iOS, ScrollView.scrollToEnd() computes its target using UIScrollView.contentInset, even when UIKit has expanded the effective scrollable area through adjustedContentInset.

This makes programmatic scrolling stop before the same end position a user can reach by dragging. The issue reproduces in the official React Native reproducer template on React Native 0.86.0 with the New Architecture. It does not require Expo, navigation, a keyboard, or an explicit contentInset.

On an iPhone 17 Pro Max simulator, stock React Native stops at offset 364 while a manual drag reaches 398. The missing 34 points exactly match the device's adjusted bottom safe-area inset.

Source-level cause

RCTScrollViewComponentView::scrollToEnd currently constructs the target with _scrollView.contentInset.bottom or .right. UIKit's safe-area and bar adjustments are represented by _scrollView.adjustedContentInset, so the command omits the system-provided part of the scrollable range.

Current source:

CGFloat offsetY = _scrollView.contentSize.height - _scrollView.bounds.size.height + _scrollView.contentInset.bottom;

Using adjustedContentInset makes the command reach the same end as a manual drag.

Relationship to #56189

This is related to, but not fixed by, #56189.

That PR moves scrollTo clamping into scrollToOffset and clamps against adjustedContentInset. I built React Native from source with that PR's effective Fabric logic:

Action with #56189 logic Final vertical offset
scrollTo({y: 9999}) 398 (fixed)
scrollToEnd() 364 (still incorrect)

scrollToEnd() constructs an already-valid target of 364 before calling scrollToOffset, so later clamping correctly leaves that target unchanged. Changing only scrollToEnd() to construct its target from adjustedContentInset makes it reach 398.

Steps to reproduce

  1. Clone the mandatory reproducer, which starts from react-native-community/reproducer-react-native at React Native 0.86.0.
  2. Run cd ReproducerApp && yarn install.
  3. Run cd ios && pod install && cd ...
  4. Run yarn ios on an iOS simulator with a bottom safe area.
  5. Tap scrollToEnd.
  6. Observe offset 364 / raw 364 / adjusted 398 / safe bottom 34.
  7. Tap Reset, then drag the scroll view manually to its end.
  8. Observe that manual scrolling reaches 398.

The reproducer uses one plain vertical ScrollView:

<ScrollView
  ref={scrollViewRef}
  contentInsetAdjustmentBehavior="always"
>
  {/* content taller than the viewport */}
</ScrollView>

and invokes:

scrollViewRef.current?.scrollToEnd({animated: false});

React Native Version

0.86.0

Affected Platforms

Runtime - iOS

Areas

Fabric - The New Renderer

Output of npx @react-native-community/cli info

System:
  OS: macOS 26.5.1
  CPU: (12) arm64 Apple M4 Pro
  Memory: 133.78 MB / 24.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 25.5.0
    path: /opt/homebrew/bin/node
  Yarn: Not Found
  npm:
    version: 11.8.0
    path: /opt/homebrew/bin/npm
  Watchman:
    version: 2026.01.12.00
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.16.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 25.5
      - iOS 26.5
      - macOS 26.5
      - tvOS 26.5
      - visionOS 26.5
      - watchOS 26.5
  Android SDK: Not Found
IDEs:
  Android Studio: 2025.1 AI-251.26094.121.2513.14007798
  Xcode:
    version: 26.6/17F113
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.16
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  react:
    installed: 19.2.3
    wanted: 19.2.3
  react-native:
    installed: 0.86.0
    wanted: 0.86.0
iOS:
  New Architecture: enabled by the official reproducer template

Stacktrace or Logs

There is no crash. The deterministic runtime measurements are:

Native implementation scrollToEnd() scrollTo({y: 9999}) Manual end
Stock React Native 0.86.0 364 364 398
#56189 effective Fabric logic 364 398 398
#56189 logic plus the scrollToEnd fix 398 398 398

All native variants were compiled from source before testing.

MANDATORY Reproducer

https://github.com/eliotgevers/react-native-scroll-to-end-adjusted-inset-repro

Screenshots and Videos

Stock React Native 0.86.0 Patched native source
scrollToEnd() stops at 364 and cuts off the true end marker. scrollToEnd() reaches the adjusted end at 398.
Stock React Native scrollToEnd stops at offset 364 Patched React Native scrollToEnd reaches offset 398

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Đọc RCTScrollViewComponentView::scrollToEnd và so sánh phép tính đích của nó với hành vi inset đã được điều chỉnh được mô tả trong issue. Chạy reproducer bắt buộc trên trình mô phỏng iOS, sau đó xác minh rằng scrollToEnd đạt đến cùng offset cuối như thao tác kéo thủ công, bao gồm cả inset của safe area phía dưới.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
ios, objective-c
Lĩnh vực
mobile-dev
Loại issue
Lỗi
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Ít trao đổi
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
78/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.