Make no-op IAccessibilityNotifier for ConPTY mode #14409

Closed
opened 2026-01-31 04:09:31 +00:00 by claunia · 2 comments
Owner

Originally created by @miniksa on GitHub (Jul 6, 2021).

Originally assigned to: @miniksa on GitHub.

@DHowett I finished the first part and moved the check inside NotifyAccessibilityEventing. I found that I'm not the right person to do the second part: add a new empty/no-op IAccessibilityNotifier and move everything to the no-op one. There's a lot legacy conhost & internal Windows stuff that I don't understand enough to modify. I figure I will just limit this PR to the first part. It's still brings noticeable performance benefit (updated my comment above). That is, in fact, my initial intention for this PR.

Originally posted by @skyline75489 in https://github.com/microsoft/terminal/issues/10537#issuecomment-872635147

Originally created by @miniksa on GitHub (Jul 6, 2021). Originally assigned to: @miniksa on GitHub. @DHowett I finished the first part and moved the check inside `NotifyAccessibilityEventing`. I found that I'm not the right person to do the second part: add a new empty/no-op IAccessibilityNotifier and move everything to the no-op one. There's a lot legacy conhost & internal Windows stuff that I don't understand enough to modify. I figure I will just limit this PR to the first part. It's still brings noticeable performance benefit (updated my comment above). That is, in fact, my initial intention for this PR. _Originally posted by @skyline75489 in https://github.com/microsoft/terminal/issues/10537#issuecomment-872635147_
Author
Owner

@ghost commented on GitHub (Jul 14, 2021):

:tada:This issue was addressed in #10569, which has now been successfully released as Windows Terminal v1.9.1942.0.🎉

Handy links:

@ghost commented on GitHub (Jul 14, 2021): :tada:This issue was addressed in #10569, which has now been successfully released as `Windows Terminal v1.9.1942.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.9.1942.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Author
Owner

@ghost commented on GitHub (Jul 14, 2021):

:tada:This issue was addressed in #10569, which has now been successfully released as Windows Terminal Preview v1.10.1933.0.🎉

Handy links:

@ghost commented on GitHub (Jul 14, 2021): :tada:This issue was addressed in #10569, which has now been successfully released as `Windows Terminal Preview v1.10.1933.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.10.1933.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#14409