sveltejs / svelte

Cybernetically enhanced web apps
https://svelte.dev
MIT License
76.98k stars 4k forks source link

fix: 11623 - Binding scrollY to svelte:window causes implicit scrolling #11630

Closed adiguba closed 2 weeks ago

adiguba commented 2 weeks ago

<svelte:window bind:scrollY /> force the scroll to 0, because scrollY is undefined. This should be ignored, so the browser can maintain scrolling on refresh.

Fix https://github.com/sveltejs/svelte/issues/11623

Before submitting the PR, please make sure you do the following

Tests and linting

changeset-bot[bot] commented 2 weeks ago

⚠️ No Changeset found

Latest commit: ed6ea36137916ab5a68c055385ceeb6d0a66f1b5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR