[iOS] UIKit's scroll to a focused field can leave the pager between pages
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 52/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- ios, react-native, typescript
Rechercherichtung
Start with the example app screen “Issue #1154 Page Change Focus Repro” and set styles.events to height: 44 to reproduce the timing. Trace the iOS pager handling of interrupted setPage calls alongside the focus and keyboard events, then run the described release-build sequence. Done means the pager settles on page 2, reports matching selection and scroll state, and the focused input and keyboard behave consistently.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Environment
react-native-pager-view:main- React Native 0.88.0, new architecture
- iPhone 18 Pro simulator, iOS 27.0, release build
Description
When a text field on another page takes focus, UIKit scrolls the pager's collection view to reveal it: scrollRectToVisible, twice and animated, once while the field becomes first responder and once from the keyboard notification. That scroll replaces the slide TabView is running and stops at the smallest offset that shows the field, and UIKit leaves the pager there.
The selection and the screen then disagree. onPageSelected reports page 2 while the pager rests nearest page 1, the log reads selected 2 and then state idle at 1, and the screen shows a sliver of page 0 beside page 1. Because the pager is not on a page boundary nothing reports a rest, so the input on the page being left keeps focus with the keyboard open.
Measuring the collection view's offset at scrollViewDidEndScrollingAnimation, against a 402 pt page:
| run | end offset |
|---|---|
| interrupt, with the arriving page's input focused in the same handler | 377 |
| interrupt, no focus | 804 |
| interrupt, keyboard already open from the previous page | 804 |
focus on arrival, single setPage, no interrupt |
402 |
Both ingredients are needed, and 377 is not arbitrary: it is the least the pager can scroll to bring that field on screen.
A per-instance override is viable, since PagerGestureObserver.installGestureArbitration already gives the pager's collection view its own runtime subclass. It is a behaviour change: an app that focuses an input on a page it is not showing would see nothing move, where today the pager is dragged between pages instead. An enclosing scroll view is unaffected, since in a pager inside a ScrollView inside a KeyboardAvoidingView the pager receives no reveal at all in 6 of 6 runs, and every one goes to the outer scroll view and the field's own editor.
Reproducible Demo
The example app has the screen already: Issue #1154 Page Change Focus Repro. Its event log reflows as it grows, which shifts the timing, so give the events box a fixed height first (height: 44 on styles.events) to see this reliably.
- Open the screen in a release build
- Tap onPress, so the arriving page's input is focused in the same handler as the command
- Tap the input on page 0, so the keyboard opens
- Tap Go 1 then 2, which sends
setPage(1)and thensetPage(2)fromonPageSelected - The pager stops between pages and stays there. The status line reads
Selected: 2, the log endsstate idle at 1, and the input on page 1 is still focused with the keyboard open
With that fixed height, two runs in six end this way on main. Without it the screen's own timing hides it most of the time, though the same case failed CI on #1224 with Selected: 1 | Focused: input 1 | Keyboard: visible | Scrolls: 9.
A fix is open as #1234, which declines the reveal from iOS 16 rather than repairing the landing after it.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 3.4k
- Forks
- 491
- Ø Merge
- 2 T. 4 Std.
- Gemergte PRs (30 T.)
- 51
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus callstack/react-native-pager-view
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
callstack/react-native-pager-view#1232 · 2 Reaktionen ·
Maintainer antworten meist innerhalb von 1 Tag
-
[iOS] setPage does not animate when the pager resizes with the keyboardEvtl. vergeben @TaduJR hat das vor 4 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 52/100
callstack/react-native-pager-view#1223 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Resolution: Needs Repro
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 32/100
callstack/react-native-pager-view#1222 · 2 Kommentare · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 58/100
callstack/react-native-pager-view#1189 ·
Maintainer antworten meist innerhalb von 1 Tag
-
iOS: PagerScrollDelegate proxies to itself and overflows the stack when the weak originalDelegate diesEvtl. vergeben @lvlrSajjad hat das vor 56 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
callstack/react-native-pager-view#1146 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in callstack/react-native-pager-view
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
earendil-works/pi-claude-code#36 ·
Maintainer antworten meist innerhalb von 6 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
Maintainer antworten meist innerhalb von 1 Tag
-
Resources: New work of jabodetabekEvtl. vergeben @rmt-svc hat das heute übernommen. Offenresources
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
railmapgen/rmp-gallery#4167 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
documentation p1
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 83/100
microsoft/monaco-editor#5577 ·
Maintainer antworten meist innerhalb von 1 Tag