Skip to content

UI Automation in Windows Console: disable filtering when tab is pressed#9936

Closed
codeofdusk wants to merge 1 commit into
nvaccess:masterfrom
codeofdusk:cmduia8-tabs
Closed

UI Automation in Windows Console: disable filtering when tab is pressed#9936
codeofdusk wants to merge 1 commit into
nvaccess:masterfrom
codeofdusk:cmduia8-tabs

Conversation

@codeofdusk

@codeofdusk codeofdusk commented Jul 16, 2019

Copy link
Copy Markdown
Contributor

Link to issue number:

Related to #9614. Related to, but does not fix, #6095.

Summary of the issue:

When pressing tab in UIA consoles, very short completions are not read.

Description of how this pull request fixes the issue:

When tab is pressed, a new self._hasTab flag on UIA consoles is set to True, causing the filtering logic in _reportNewText to be temporarily disabled.

Testing performed:

On Windows 10 1903 in a UIA console:

  1. Created a directory called testenv, containing two subdirectories w14 and w15.
  2. Changed to the test directory, and typed cd w tab

Before the change: NVDA said nothing.
After the change: NVDA said "w14".

This PR does not affect legacy consoles.

Known issues with pull request:

None.

Change log entry:

None.

codeofdusk added a commit to codeofdusk/nvda that referenced this pull request Jul 16, 2019
@codeofdusk

This comment has been minimized.

@codeofdusk codeofdusk changed the base branch from beta to master July 17, 2019 16:14
@feerrenrut feerrenrut self-requested a review July 26, 2019 13:23
@feerrenrut

Copy link
Copy Markdown
Contributor

I won't merge this yet. @codeofdusk first can you confirm if there is any interdependency between these Pull requests?

@codeofdusk

Copy link
Copy Markdown
Contributor Author

This was intended as a backported change from #9915 to beta, which is no longer relevant.

@codeofdusk codeofdusk closed this Jul 27, 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.

2 participants