Add sound split disabled mode

[mltony]

Link to issue number:

Closes #16287

Summary of the issue:

Adding disabled sound split mode

Description of user facing changes

Adding disabled sound split mode

Description of development approach

1. If sound split mode is disabled at startup, we're not touching any volumes.
2. If user toggles to disabled mode, then first we set mode to NVDA_BOTH_APPS_BOTH to effectively restore the volumes, then set sound pslit state to off.

Testing strategy:

Manual

Known issues with pull request:

Code Review Checklist:

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

[AppVeyorBot]

• FAIL: Translation comments check. Translation comments missing or unexpectedly included. See build log for more information.
• PASS: Unit tests.
• PASS: Lint check.
• PASS: System tests (tags: installer NVDA).
• Build (for testing PR): https://ci.appveyor.com/api/buildjobs/ikho7bhl85lop5pj/artifacts/output/nvda_snapshot_pr16331-31517,6c81d764.exe
• CI timing (mins):
  INIT 0.0,
  INSTALL_START 1.0,
  INSTALL_END 0.9,
  BUILD_START 0.0,
  BUILD_END 28.0,
  TESTSETUP_START 0.0,
  TESTSETUP_END 0.4,
  TEST_START 0.0,
  TEST_END 2.3,
  FINISH_END 0.2

See test results for failed build of commit 6c81d76403

[cary-rowen]

Hi @mltony

Thank you for your work.

I tested this snapshot,
I first selected "NVDA on the left and applications on the right" in the "Audio" settings panel, clicked OK, and NVDA performed the expected.

Then, I slected "Sound split disabled" in combo box in the "audio" settings panel and click "OK" did not disable sound splitt.

However, when I use the gesture nvda+alt+S it can be disabled successfully.

Thanks

[mltony]

@cary-rowen, just fixed that, please try again.

[LeonarddeR]

Thanks for taking this

[LeonarddeR]

Personally, I think the difference between off and NVDA in both channels and applications in both channels is a bit vague at first sight. The documentation definitely needs updating, but may be debate about the UX first? @Qchristensen What do you think?