Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

virtual-core: a clamp retry can overwrite a scroll command and ignores _canWriteScroll()

Geschlossen
  • Stimmt etwas nicht?
#1,327 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
25/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
react, typescript
Bereich
frontend, performance

Rechercherichtung

Start in the virtual-core scroll write path: _retryClampedAdjustment, _canWriteScroll, _startScroll, _writeAwaitingCommand, and the _willUpdate and resizeItem calls that grow the sizer. Reproduce the case where a clamped retry at { target: 4850, maxAtWrite: 4800 } fires after a scrollToIndex write to 550. Done when a command's target is never overwritten by a retry, the retry respects _canWriteScroll(), and tests cover desktop and the iOS 27 tail timer.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Problem

_retryClampedAdjustment repeats a compensation write that the browser clamped because the sizer was not big enough yet (#1258). It has two problems.

1. The retry can overwrite a scroll command
  1. A hold flush or a compensation write is clamped. The virtualizer records a retry, for example { target: 4850, maxAtWrite: 4800 }.
  2. A scroll command writes. For example, scrollToIndex writes 550.
  3. _willUpdate or resizeItem runs before the command's read-back. The sizer is now bigger, so the retry writes 4850.

The scroll position is 4850, not the target of the command.

  • This occurs on desktop and on iOS.
  • On iOS 27 it occurs more often. The tail timer flushes the hold and then writes a queued command in the same task.
  • It occurs only with adapters whose notify does not grow the sizer synchronously. The React adapter grows it synchronously.
  • #1324 fixed this for a queued command: _writeAwaitingCommand clears the retry. Other commands still have the problem.
2. The retry ignores _canWriteScroll()

The retry can write during the iOS touch tail or under a finger that drags. iOS ignores a write under a finger. On iOS 26 and earlier, a write during the tail stops the momentum.

Possible fix

  • Clear _clampedAdjustment when a command starts (_startScroll), or skip the retry while scrollState !== null.
  • In _retryClampedAdjustment, return early when _canWriteScroll() is false. Keep the record for the next opportunity.
  • Add tests for desktop and for the iOS 27 tail timer.

Found during the review of #1324. Related: #1258, #1307.

Vorherrschende Sprache
TypeScript
Sterne
7.2k
Forks
479
Ø Merge
1 T. 2 Std.
Gemergte PRs (30 T.)
34

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus TanStack/virtual

Alle Issues in TanStack/virtual

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.