Skip to content

fix(site): reset scroll position when navigating between docs - #82

Merged
Chenglong-MS merged 10 commits into
microsoft:devfrom
yelper:dev/alsarika/fix-docs-scroll-position
Aug 5, 2026
Merged

fix(site): reset scroll position when navigating between docs#82
Chenglong-MS merged 10 commits into
microsoft:devfrom
yelper:dev/alsarika/fix-docs-scroll-position

Conversation

@yelper

@yelper yelper commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

When navigating between pages in the documentation site, the scroll position is not reset to top as in normal page navigation.

This PR identifies hash changes that actually resulted in a different page content being loaded and explicitly sets the scrollPosition of the content to the top of the document.

spboyer and others added 10 commits July 15, 2026 17:18
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5b6d0c5f-588a-4dc5-86de-99a1c8eb323e
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5b6d0c5f-588a-4dc5-86de-99a1c8eb323e
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5b6d0c5f-588a-4dc5-86de-99a1c8eb323e

Copilot-Session: 693921d5-fb57-4a87-8c08-dca47c24aa76
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5b6d0c5f-588a-4dc5-86de-99a1c8eb323e

Copilot-Session: 693921d5-fb57-4a87-8c08-dca47c24aa76
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5b6d0c5f-588a-4dc5-86de-99a1c8eb323e

Copilot-Session: 693921d5-fb57-4a87-8c08-dca47c24aa76
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5b6d0c5f-588a-4dc5-86de-99a1c8eb323e

Copilot-Session: 693921d5-fb57-4a87-8c08-dca47c24aa76
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5b6d0c5f-588a-4dc5-86de-99a1c8eb323e

Copilot-Session: 693921d5-fb57-4a87-8c08-dca47c24aa76
The documentation page uses a custom scroll container, so the browser's
default scroll-to-top on navigation doesn't apply. Add a useEffect that
scrolls the container to the top whenever the active doc slug changes,
unless a heading anchor is pending.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@Chenglong-MS
Chenglong-MS changed the base branch from main to dev August 5, 2026 07:13
@Chenglong-MS
Chenglong-MS merged commit 0d3abfb into microsoft:dev Aug 5, 2026
7 checks passed
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