[PR #4525] [MERGED] uia: add support for scrolling the viewport #25814

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/4525
Author: @carlos-zamora
Created: 2/11/2020
Status: ✅ Merged
Merged: 2/11/2020
Merged by: @DHowett-MSFT

Base: master ← Head: dev/cazamor/acc/utr-scroll


📝 Commits (1)

  • 033f0db Add support for scrolling the viewport in accessibility

📊 Changes

5 files changed (+11 additions, -3 deletions)

View changed files

📝 src/cascadia/TerminalControl/TermControlUiaProvider.cpp (+5 -0)
📝 src/cascadia/TerminalControl/TermControlUiaProvider.hpp (+1 -0)
📝 src/cascadia/TerminalControl/UiaTextRange.cpp (+3 -2)
📝 src/interactivity/win32/screenInfoUiaProvider.hpp (+1 -1)
📝 src/types/ScreenInfoUiaProviderBase.h (+1 -0)

📄 Description

Summary of the Pull Request

The UIA Provider now scrolls the viewport when necessary. This just fills in the missing virtual function in Terminal to have the same behavior as what it does in ConHost.

PR Checklist

Detailed Description of the Pull Request / Additional comments

ChangeViewport is now a virtual function at the ScreenInfoUiaProvider layer to have access to the TermControl.

In ConHost, we pass this call up to the WindowUiaProvider layer. In Terminal, we don't need to do that because the concept of updating the viewport is handled at the TermControl layer. So we just call that function and voila!

Validation Steps Performed

Verified using Accessibility Insights and Narrator.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/4525 **Author:** [@carlos-zamora](https://github.com/carlos-zamora) **Created:** 2/11/2020 **Status:** ✅ Merged **Merged:** 2/11/2020 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/cazamor/acc/utr-scroll` --- ### 📝 Commits (1) - [`033f0db`](https://github.com/microsoft/terminal/commit/033f0dbbedd6668e056777b2dc6ab81d20b450ba) Add support for scrolling the viewport in accessibility ### 📊 Changes **5 files changed** (+11 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalControl/TermControlUiaProvider.cpp` (+5 -0) 📝 `src/cascadia/TerminalControl/TermControlUiaProvider.hpp` (+1 -0) 📝 `src/cascadia/TerminalControl/UiaTextRange.cpp` (+3 -2) 📝 `src/interactivity/win32/screenInfoUiaProvider.hpp` (+1 -1) 📝 `src/types/ScreenInfoUiaProviderBase.h` (+1 -0) </details> ### 📄 Description ## Summary of the Pull Request The UIA Provider now scrolls the viewport when necessary. This just fills in the missing virtual function in Terminal to have the same behavior as what it does in ConHost. ## PR Checklist * [X] Closes #2361 * [X] CLA signed. ## Detailed Description of the Pull Request / Additional comments `ChangeViewport` is now a virtual function at the `ScreenInfoUiaProvider` layer to have access to the TermControl. In ConHost, we pass this call up to the WindowUiaProvider layer. In Terminal, we don't need to do that because the concept of updating the viewport is handled at the TermControl layer. So we just call that function and _voila_! ## Validation Steps Performed Verified using Accessibility Insights and Narrator. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:11:57 +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#25814