Skip to content

fix scroll behaviour#1131

Merged
deer-wmde merged 2 commits into
mainfrom
de/fix-scrolling
Jun 11, 2026
Merged

fix scroll behaviour#1131
deer-wmde merged 2 commits into
mainfrom
de/fix-scrolling

Conversation

@deer-wmde

Copy link
Copy Markdown
Contributor

This change came up with #1128 and I put it in its own PR because #1130 also needs this

This fix addresses several things:

  1. scroll to the top of the page after following an internal page link
  2. making on-page anchor-links (#) work specified with vue router-link

https://phabricator.wikimedia.org/T408254
https://phabricator.wikimedia.org/T420575

@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown

Deployment previews on netlify for branch refs/pull/1131/merge will be at the following locations (when build is done):

@rosalieper rosalieper left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tried it and it works!

Comment thread src/router/index.js
@deer-wmde deer-wmde merged commit 5dbbcb0 into main Jun 11, 2026
7 checks passed
@deer-wmde deer-wmde deleted the de/fix-scrolling branch June 11, 2026 09:19
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.

3 participants