Skip to content
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

UI Automation in Windows Console: add focus redirection for the UIA console main window #9674

Merged
merged 3 commits into from
Jun 11, 2019

Conversation

codeofdusk
Copy link
Contributor

@codeofdusk codeofdusk commented Jun 5, 2019

Link to issue number:

Builds on #9614.

Summary of the issue:

Currently, in UIA consoles:

  • Sometimes, attempting to interact with the console too quickly after focusing the window can make NVDA unable to get any caret or review information or receive new text events.

Description of how this pull request fixes the issue:

This PR adds focus redirection from the console application's window to its text area.

Testing performed:

On Windows 10 1903 with UIA console support enabled:

  1. Open a command console.
  2. Press alt+space.
  3. Press e to open the edit menu.
  4. Press p to paste.

Before this change: NVDA became unable to interact with the console.
After this change: NVDA's console interaction was unaffected.

Known issues with pull request:

None known.

Change log entry:

None.

@codeofdusk
Copy link
Contributor Author

def findExtraOverlayClasses(obj, clsList):
if obj.UIAElement.cachedAutomationId == "Text Area":
clsList.append(winConsoleUIA)
if obj.UIAElement.cachedAutomationId == "Console Window":
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make this an elif

@michaelDCurran michaelDCurran merged commit 11ec1be into nvaccess:master Jun 11, 2019
@nvaccessAuto nvaccessAuto added this to the 2019.3 milestone Jun 11, 2019
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.

4 participants