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

Attempt to cancel speech for expired focus events: dialog title discarded when finding text in browse mode #11144

Closed
LeonarddeR opened this issue May 11, 2020 · 6 comments · Fixed by #11245

Comments

@LeonarddeR
Copy link
Collaborator

Steps to reproduce:

  1. Enable Attempt to cancel speech for expired focus events (set it to yes)
  2. Open a browse mode document, e.g. in Firefox
  3. Press Ctrl+NVDA+f to open the NVDA find dialog.

Actual behavior:

NVDA only announces:

Type the text you wish to find  edit

Find dialog is shown by the speech viewer, but it is interrupted immediately.

Expected behavior:

NVDA announces:

Find  dialog
Type the text you wish to find  edit

System configuration

NVDA installed/portable/running from source:

Installed

NVDA version:

alpha-20175,9967903a

Windows version:

Windows 10 2004 19041.208

Name and version of other software in use when reproducing the issue:

Firefox 76.0.1

Other questions

Does the issue still occur after restarting your computer?

Yes

Have you tried any other versions of NVDA? If so, please report their behaviors.

NO

If addons are disabled, is your problem still occuring?

Yes

Did you try to run the COM registry fixing tool in NVDA menu / tools?

No, n/a

feerrenrut added a commit that referenced this issue Jun 16, 2020
fixes #11132 - Say-all repeats some lines
fixes #11144 - Attempt to cancel speech for expired focus events: dialog title discarded when finding text in browse mode

Adds a speechManager test harness
Logging: Support writing unit tests by enabling 'IS_UNIT_TEST_LOG_ENABLED'.
Logging: Improve repr for _CancellableSpeechCommand, include devInfo for 'isValid'
Logging: Help debug double speaking
Tests: for double speaking during say-all.
Tests: Wrapped index (IndexCommands) comparisons and fixes comparisons
Tests: From manual testing during initial development of SpeechManager
Tests: Cancellable Speech
Fix: double speaking error
Fix: sudden silence during cancellable speech error which is broken during "Fix double speaking error"
Fix: cancelling speech on index callback
Fix: missing speech in run dialog
 - Speech is valid if ancestor of current focus.
 - Only ask speechManager to remove cancelled speech after the isValidDeps are updated,
    specifically lastQueuedFocusObject
@nvaccessAuto nvaccessAuto added this to the 2020.2 milestone Jun 16, 2020
@lukaszgo1
Copy link
Contributor

Hi @feerrenrut I'm afraid this issue is not fixed for me with NVDA version alpha-20339,975d39d1 and Firefox Nightly 79 - dialog title is still discarded when it opens. @LeonarddeR Can you confirm?

@feerrenrut
Copy link
Contributor

yes I can still reproduce this with the NVDA find dialog. I was using the run dialog (windows key + r) as an example when attempting to fix this issue. The issue here is that the title does not seem to be a direct focus ancestor of the edit control, at least that is what the "speechManager" debug logging says (enabled via advanced panel).

@feerrenrut feerrenrut reopened this Jun 17, 2020
@feerrenrut
Copy link
Contributor

The run dialog, however, does seem to be fixed.

@feerrenrut
Copy link
Contributor

This should be fixed now, can anyone confirm?

@lukaszgo1
Copy link
Contributor

cannot reproduce with NVDA version alpha-21348,6af30832.

@feerrenrut
Copy link
Contributor

Thanks @lukaszgo1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants