[PR #16261] Fix scrollbar resetting position on save #30880

Open
opened 2026-01-31 09:43:32 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/16261

State: closed
Merged: Yes


Summary of the Pull Request

This PR fixes Issue #11875 by introducing a ScrollViewer and some logic for the scrollbar.

Detailed Description of the Pull Request / Additional comment

The ScrollViewer prevents the scrollbar from scrolling to the top whenever "Save" is clicked in the Settings. In addition, the scrollbar is scrolled to the top of the page whenever navigating to another page within Settings. The scrollbar will not reset if attempting to navigate to the same page that is already navigated to.

Validation Steps Performed

Manual testing of the Settings by building the Terminal app.

PR Checklist

**Original Pull Request:** https://github.com/microsoft/terminal/pull/16261 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request This PR fixes Issue #11875 by introducing a ScrollViewer and some logic for the scrollbar. ## Detailed Description of the Pull Request / Additional comment The ScrollViewer prevents the scrollbar from scrolling to the top whenever "Save" is clicked in the Settings. In addition, the scrollbar is scrolled to the top of the page whenever navigating to another page within Settings. The scrollbar will not reset if attempting to navigate to the same page that is already navigated to. ## Validation Steps Performed Manual testing of the Settings by building the Terminal app. ## PR Checklist - [x] Closes #11875 - [ ] Tests added/passed - [ ] Documentation updated - If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx - [ ] Schema updated (if necessary)
claunia added the pull-request label 2026-01-31 09:43:32 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#30880