Skip to content

Fix page scrolling to bottom in responsive mobile layout - #3478

Merged
rosslovas merged 1 commit into
mainfrom
ross/fix-mobile-scroll
Sep 25, 2026
Merged

rosslovas merged 1 commit into
mainfrom
ross/fix-mobile-scroll

Conversation

@rosslovas

@rosslovas rosslovas commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

In #3432 we made the page automatically scroll the current page in the nav sidebar into view, which has been a really nice usability improvement. However, it didn't account for the responsive / narrow layout: there, it would scroll the document itself to the bottom, unconditionally.

This PR fixes that by specifically targeting the scroll to the nav sidebar in the wide layout / the nav menu, when opened, in the narrow layout.

Before

Page scrolls to the bottom on load, nav menu is not scrolled at all

Screen.Recording.2026-09-25.at.9.10.31.am.mov

After

Page is not erroneously scrolled, nav menu is scrolled such that the current page is in view

Screen.Recording.2026-09-25.at.9.22.13.am.mov

@team-marketing-branch-protections

Copy link
Copy Markdown

Pull request environment is available at https://stoctodocspr3478.z22.web.core.windows.net/.

You can view the ephemeral environment status in Octopus Deploy.

This environment will be automatically deprovisioned when the pull request is closed, or after 7 days of inactivity.

@rosslovas
rosslovas merged commit 64748e6 into main Sep 25, 2026
9 checks passed
@rosslovas
rosslovas deleted the ross/fix-mobile-scroll branch September 25, 2026 00:10
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