Skip to content

Fallback to auto synth instead of silence if synth change failed#15119

Merged
seanbudd merged 3 commits into
masterfrom
fallbackToSynth
Jul 14, 2023
Merged

Fallback to auto synth instead of silence if synth change failed#15119
seanbudd merged 3 commits into
masterfrom
fallbackToSynth

Conversation

@seanbudd

@seanbudd seanbudd commented Jul 11, 2023

Copy link
Copy Markdown
Member

Link to issue number:

Fixes #15096

Summary of the issue:

When an active profile has the synthesizer set to silence, and a user switches profiles to a synthesizer which fails to load, NVDA fails back to the silence synthesizer.

This is because the default behaviour on failure is to roll back to the previous synthesizer, which is appropriate except for silence.

As this is an attempt to change from silence to speech, instead NVDA should fallback to the next synthesizer, rather than the previous synthesizer.

Description of user facing changes

NVDA falls back to the silence synthesizer when:

  • a user is using the silence synthesizer,
  • and attempts to switch synthesizers e.g. via a profile change
  • and the synthesizer fails to load

Description of development approach

ensure the previous synthesizer we are switching back to is not silence.
Instead, the synth will switch to the next one, which may eventually reach silence if subsequent synthesizers fail to load.

Testing strategy:

Confirm with @fernando-jose-silva

Known issues with pull request:

It would be ideal for us to switch back to OneCore before cycling to the next synthesizer, however it's impossible to do this without additional synth history state tracking

Change log entries:

N/A small change

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • Security precautions taken.

@seanbudd seanbudd requested a review from a team as a code owner July 11, 2023 05:50
@seanbudd seanbudd requested a review from michaelDCurran July 11, 2023 05:50
@AppVeyorBot

Copy link
Copy Markdown

See test results for failed build of commit 61e6454aa8

@seanbudd seanbudd marked this pull request as draft July 11, 2023 06:22
@AppVeyorBot

Copy link
Copy Markdown

See test results for failed build of commit 3a1b47985f

@seanbudd seanbudd marked this pull request as ready for review July 12, 2023 01:40
@AppVeyorBot

Copy link
Copy Markdown

See test results for failed build of commit 9395e86953

@seanbudd seanbudd merged commit 6f0f312 into master Jul 14, 2023
@seanbudd seanbudd deleted the fallbackToSynth branch July 14, 2023 07:12
@nvaccessAuto nvaccessAuto added this to the 2023.2 milestone Jul 14, 2023
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.

nvda does not return voice after exiting a settings profile

4 participants