Skip to content

Commit b8cf4cf

Browse files
authored
Unbind application volume adjustment gestures (#17634)
Closes #17272 Summary of the issue: The application volume adjuster feature introduced in #16591 includes default keyboard gestures that: 1. Conflict with the default gestures used by NVDA Remote; and 2. Don't do anything by default, as the feature is disabled. Description of user facing changes These gestures are unbound by default. Users can still bind gestures to these commands from the input gestures dialog. Description of development approach Removed the `gesture` argument to the `script` decorator on `GlobalCommands.script_increaseApplicationsVolume`, `GlobalCommands.script_decreaseApplicationsVolume`, and `GlobalCommands.script_toggleApplicationsMute`. Testing strategy: Ran NVDA from source. Ensured that the commands were unbound by executing the previous keyboard gestures. Re-bound the gestures in the input gestures dialog, and ensured they worked as expected. Known issues with pull request: None.
1 parent a9a081b commit b8cf4cf

File tree

3 files changed

+6
-19
lines changed

3 files changed

+6
-19
lines changed

source/globalCommands.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4832,7 +4832,6 @@ def script_cycleSoundSplit(self, gesture: "inputCore.InputGesture") -> None:
48324832
"Increases the volume of other applications",
48334833
),
48344834
category=SCRCAT_AUDIO,
4835-
gesture="kb:NVDA+alt+pageUp",
48364835
)
48374836
def script_increaseApplicationsVolume(self, gesture: "inputCore.InputGesture") -> None:
48384837
appsVolume._adjustAppsVolume(5)
@@ -4843,7 +4842,6 @@ def script_increaseApplicationsVolume(self, gesture: "inputCore.InputGesture") -
48434842
"Decreases the volume of other applications",
48444843
),
48454844
category=SCRCAT_AUDIO,
4846-
gesture="kb:NVDA+alt+pageDown",
48474845
)
48484846
def script_decreaseApplicationsVolume(self, gesture: "inputCore.InputGesture") -> None:
48494847
appsVolume._adjustAppsVolume(-5)
@@ -4864,7 +4862,6 @@ def script_toggleApplicationsVolumeAdjuster(self, gesture: "inputCore.InputGestu
48644862
"Mutes or unmutes other applications",
48654863
),
48664864
category=SCRCAT_AUDIO,
4867-
gesture="kb:NVDA+alt+delete",
48684865
)
48694866
def script_toggleApplicationsMute(self, gesture: "inputCore.InputGesture") -> None:
48704867
appsVolume._toggleAppsVolumeMute()

user_docs/en/changes.md

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,9 @@
99
* Support for math in PDFs has been added.
1010
This works for formulas with associated MathML, such as some files generated by newer versions of TeX/LaTeX.
1111
Currently this is only supported in Foxit Reader & Foxit Editor. (#9288, @NSoiffer)
12-
* Commands to adjust the volume of other applications besides NVDA have been added.
13-
To use this feature, "allow NVDA to control the volume of other applications" must be enabled in the audio settings panel. (#16052, @mltony, @codeofdusk)
14-
* `NVDA+alt+pageUp`: Increase the volume of all other applications.
15-
* `NVDA+alt+pageDown`: Decrease the volume of all other applications.
16-
* `NVDA+alt+delete`: Mute the volume of all other applications.
12+
* The ability to adjust the volume of other applications besides NVDA has been added.
13+
To use this feature, "allow NVDA to control the volume of other applications" must be enabled in the audio settings panel. (#16052, #17634, @mltony, @codeofdusk)
14+
* Added new unassigned gestures to increase, decrease and mute the volume of all other applications.
1715
* When editing in Microsoft PowerPoint text boxes, you can now move per sentence with `alt+upArrow`/`alt+downArrow`. (#17015, @LeonarddeR)
1816
* In Mozilla Firefox, NVDA will report the highlighted text when a URL containing a text fragment is visited. (#16910, @jcsteh)
1917
* NVDA can now report when a link destination points to the current page. (#141, @LeonarddeR, @nvdaes)

user_docs/en/userGuide.md

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2489,24 +2489,16 @@ While [audio ducking](#SelectSynthesizerDuckingMode) does change the volume of o
24892489
##### Volume of other applications {#OtherAppVolume}
24902490

24912491
This slider allows you to adjust the volume of all currently running applications other than NVDA.
2492-
This volume can also be controlled via the following keyboard commands from anywhere:
24932492

2494-
| Name | Key | Description |
2495-
|---|---|---|
2496-
| Increase volume of other applications | `NVDA+alt+pageUp` | Increases the volume of all applications except NVDA. |
2497-
| Decrease volume of other applications | `NVDA+alt+pageDown` | Decreases the volume of all applications except NVDA. |
2493+
To increase or decrease the volume of all currently running applications from anywhere, please assign custom gestures using the [Input Gestures dialog](#InputGestures).
24982494

24992495
##### Mute other applications {#OtherAppMute}
25002496

25012497
This check box allows you to mute or unmute all applications except NVDA at once.
25022498

2503-
The following keyboard command can also be used from anywhere:
2504-
2505-
| Name | Key | Description |
2506-
|---|---|---|
2507-
| Mute or unmute other applications | `NVDA+alt+delete` | Toggles mute/unmute on other applications |
2499+
To mute or unmute all other applications from anywhere, please assign a custom gesture using the [Input Gestures dialog](#InputGestures).
25082500

2509-
Please note, that this option is not persistent: other apps will always be unmuted when NVDA restarts.
2501+
Please note that this option is not persistent: other apps will always be unmuted when NVDA restarts.
25102502

25112503
##### Time to keep audio device awake after speech {#AudioAwakeTime}
25122504

0 commit comments

Comments
 (0)