chore: move scroll restoration and focus reset out of client.js - #16980
Conversation
|
Install the latest version of pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/fad9a52484d020561a244623fb4e9000646e0eefOpen in |
|
fb4651b to
4386cd6
Compare
4386cd6 to
2b735ad
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: QUIET Plan: Advanced Run ID: 📒 Files selected for processing (1)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughFocus, hash lookup, and scroll management move from Merge Risk: ⚪ Minimal · up to Client focus and scroll behavior has been reorganized into dedicated runtime helpers while preserving navigation behavior, with coverage for the affected restoration and suppression paths. No current merge-blocking risk is identified. 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
Comment |
01f7ffb to
fad9a52
Compare
Scroll restoration and focus reset live in
client.jsbut read none of its state. This moves them, unchanged, into their own module, the shapesnapshots.jstook in #16685.