Skip to content

fix(react): preserve scroll authority across anchored rebuilds - #543

Merged
blove merged 4 commits into
mainfrom
blove/scroll-authority
Aug 30, 2026
Merged

fix(react): preserve scroll authority across anchored rebuilds#543
blove merged 4 commits into
mainfrom
blove/scroll-authority

Conversation

@blove

@blove blove commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • establish an explicit two-way scroll-authority protocol between the grid/DOM and row-layout controller
  • propagate controller anchor adjustments through grid state to the real scroll element before paint
  • suppress causal viewport echoes while preserving current-position behavior for semantic sort, filter, and grouping changes
  • share the row-model's exact semantic-query equality through an internal subpath
  • add controller-level and DOM integration regression coverage

Root cause

After an anchored controller rebuild, a controller status publication retriggered the viewport-feed effect. The effect treated the unchanged, pre-anchor grid offset as a fresh input and overwrote the controller's anchor-adjusted scroll position.

Verification

  • React: 117 files, 1,644 tests passed
  • row-model: 42 files, 719 tests passed
  • benchmark app: 181 tests passed
  • focused authority/navigation set: 4 files, 38 tests passed
  • all six website tests that timed out during a machine-load above 70 parallel run passed serially; the remaining three passed with one worker and a 120-second timeout
  • typecheck, public typecheck, type-performance budgets, lint, format, production build, API Extractor, publish preflight, packaging, consumer bundling, React compatibility, and security audit passed

Fixes #524

@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
pretable Ignored Ignored Aug 30, 2026 11:39pm

Request Review

@blove
blove enabled auto-merge (squash) August 30, 2026 23:39
@github-actions

Copy link
Copy Markdown
Contributor

Vercel preview ready

Preview: https://pretable-mukwr5ui9-cacheplane.vercel.app
Commit: e19cdb7945fd55474b5ce964794748deba3c2c1b

Updated automatically by the deploy-preview job.

@blove
blove merged commit 07ef7e4 into main Aug 30, 2026
21 checks passed
@blove
blove deleted the blove/scroll-authority branch August 30, 2026 23:54
This was referenced Aug 31, 2026
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.

Scroll authority: the engine's anchored scrollTop can be stomped by the viewport re-feed

1 participant