-
-
Notifications
You must be signed in to change notification settings - Fork 641
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Efficiently reviewing the output buffer in terminals #11172
Comments
For offscreen text there was #9735, but this was rejected in favour of using the native scrolling commands. |
Can we revisit #9735? Are there performance concerns if offscreen text can be reviewed? This would be really useful given how poorly the text selection keys are supported. |
Cc @feerrenrut. |
What are your thoughts on having a virtual document mode that we could fall back on using? Is that technically feasible? E.g, to solve the find problem and maybe tables. |
@codeofdusk, if #9735 is ever revived:
Additionally, a generalized "Find" mechanism is often asked for in situations other than browseable documents. While it can meet caveats in random windows, it could at least search through the same text Tables are a much more tricky matter to handle properly... |
@JulienCochuyt agreed about tables being much trickier. I only mentioned it because it can only be solved if the output buffer can be rendered in a virtual document, which could affect how this problem is solved. Having a virtual buffer would also make review and find much more pleasant. |
Related: navigation of output by command is tracked in microsoft/terminal#6232 |
It was mentioned in microsoft/terminal#6453 (comment) that some new console text can actually appear below the visible range, making it completely inaccessible to NVDA. Additionally, there are plans to automatically detect (and make clickable) links in Conhost and Terminal (see microsoft/terminal#7691). I think it makes sense to do the following:
As an initial implementation, I tried setting |
Bump. Is this still on the backlog? TO what degree does the new diffing algorithm in alpha builds contribute to this? |
I'll re-evaluate after microsoft/terminal#6986. |
OK, looks like all but 4 are fixed with #10964. |
Is 4 fixed with #12928? |
It is not, and was listed as out of scope for this issue. |
…(Windows 11 Sun Valley 2) (#10964) Supersedes #9771 and #10716. Closes #1682. Closes #8653. Closes #9867. Closes #11172. Closes #11554. Summary of the issue: Microsoft has significantly improved performance and reliability of UIA console: * microsoft/terminal#4018 is an almost complete rewrite of the UIA code which makes the console's UIA implementation more closely align with the API specification. * microsoft/terminal#10886, microsoft/terminal#10925, and microsoft/terminal#11253 form a robust testing methodology for the UIA implementation, including bug fixes in response to newly added tests based on Word's UIA implementation. * microsoft/terminal#11122 removes the thousands of empty lines at the end of the console buffer, significantly improving performance and stability. Since all console text ranges are now within the text buffer's bounds, it is no longer possible for console to crash due to the manipulation by UIA clients of an out-of-bounds text range. * Countless other accessibility-related PRs too numerous to list here. We should enable UIA support on new Windows Console builds by default for performance improvement and controllable password suppression. Description of how this pull request fixes the issue: This PR: * Exposes all three options for the UIA console feature flag in the UI (replaces the UIA check box with a combo box). * Adds a runtime check to test if `apiLevel >= FORMATTED`, and use UIA in this case when the user preference is auto. This is the case on Windows 11 Sun Valley 2 (SV2) available now in beta and set for release in the second half of 2022.
Some of the scenarios in the issue hasn't been addressed. For example, the ability to efficiently find and navigate to matches of search terms. |
Ctrl+f now works (it didn't in legacy), at least to find an initial match. F3 to move to the next match doesn't seem to work for me, but that's a Microsoft issue (i.e. out of scope for NVDA). |
CC: @codeofdusk
Is your feature request related to a problem? Please describe.
I couldn't figure out how to do the following:
Describe the solution you'd like
These are just some ideas for now, I'd love to hear how others have solved these issues.
Describe alternatives you've considered
I have therefore been forced to copy all text and paste it into a blank notepad document just to do some of these basic tasks.
The text was updated successfully, but these errors were encountered: