chore: give scroll and focus their own state and tests - #16983
chore: give scroll and focus their own state and tests#16983Nic-Polumeyv wants to merge 1 commit into
Conversation
|
Install the latest version of pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/889a45281340aad710c21b4dc98d002cb5f11b43Open in |
🦋 Changeset detectedLatest commit: 889a452 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
e67a410 to
8081c8a
Compare
8081c8a to
889a452
Compare
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
On top of #16980:
scroll.jsowns the after-navigation scroll andfocus.jsowns the blur and the focus reset, withfinish_navigationthe one place that sequences them. Whether the app moved focus during the update is now read fromdocument.activeElementdirectly instead of a snapshot taken before the update, and a navigation that does not reset no longer scrolls to where it already is.A focused SVG element is now blurred before the DOM update like any HTML element.