Skip to content

Commit eb162c0

Browse files
CyrilleB79CyrilleB79
authored
Merge 89b5f40 into a380b6a 
2 parents a380b6a + 89b5f40 commit eb162c0

3 files changed

Lines changed: 35 additions & 2 deletions

File tree

‎source/mathPres/mathPlayer.py‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# A part of NonVisual Desktop Access (NVDA)
33
# This file is covered by the GNU General Public License.
44
# See the file COPYING for more details.
5-
# Copyright (C) 2014-2022 NV Access Limited
5+
# Copyright (C) 2014-2023 NV Access Limited, Cyrille Bougot
66

77
"""Support for math presentation using MathPlayer 4.
88
"""
@@ -20,6 +20,7 @@
2020
from keyboardHandler import KeyboardInputGesture
2121
import braille
2222
import mathPres
23+
import config
2324

2425
from speech.commands import (
2526
PitchCommand,
@@ -68,7 +69,10 @@ def _processMpSpeech(text, language):
6869
if m.lastgroup == "break":
6970
out.append(BreakCommand(time=int(m.group("break")) * breakMulti))
7071
elif m.lastgroup == "char":
71-
out.extend((CharacterModeCommand(True), m.group("char"), CharacterModeCommand(False)))
72+
if config.conf["speech"][synth.name]["useSpellingFunctionality"]:
73+
out.extend((CharacterModeCommand(True), m.group("char"), CharacterModeCommand(False)))
74+
else:
75+
out.append(m.group("char"))
7276
elif m.lastgroup == "comma":
7377
out.append(BreakCommand(time=100))
7478
elif m.lastgroup in PROSODY_COMMANDS:

‎source/speech/shortcutKeys.py‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@
1111

1212
import characterProcessing
1313
from logHandler import log
14+
from synthDriverHandler import getSynth
15+
import config
1416

1517
from .commands import CharacterModeCommand
1618
from .types import SpeechSequence
@@ -79,6 +81,10 @@ def _getKeyboardShortcutSpeech(keyboardShortcut: str) -> SpeechSequence:
7981
return seq
8082

8183

84+
def shouldUseSpellingFunctionality() -> bool:
85+
synth = getSynth()
86+
return config.conf["speech"][synth.name]["useSpellingFunctionality"]
87+
8288
def _getKeySpeech(key: str) -> SpeechSequence:
8389
"""Gets the speech sequence for a string describing a key.
8490
@param key: the key string.
@@ -91,6 +97,8 @@ def _getKeySpeech(key: str) -> SpeechSequence:
9197
keySymbol = characterProcessing.processSpeechSymbol(locale, key)
9298
if keySymbol != key:
9399
return [keySymbol]
100+
if not shouldUseSpellingFunctionality():
101+
return [key]
94102
return [
95103
CharacterModeCommand(True),
96104
key,

‎tests/unit/test_speechShortcutKeys.py‎

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,16 +7,19 @@
77
"""
88

99
import unittest
10+
from unittest.mock import patch
1011

1112
from speech.shortcutKeys import (
1213
_getKeyboardShortcutSpeech,
1314
getKeyboardShortcutsSpeech,
1415
)
16+
import speech.shortcutKeys # noqa F401 - Used by unittest.mock.patch
1517
from speech.commands import CharacterModeCommand
1618

1719

1820
class Test_getKeyboardShortcutSpeech(unittest.TestCase):
1921

22+
@patch('speech.shortcutKeys.shouldUseSpellingFunctionality', lambda: True)
2023
def test_simpleLetterKey(self):
2124
"""A shortcut consisting in only one letter."""
2225

@@ -30,6 +33,21 @@ def test_simpleLetterKey(self):
3033
)
3134
self.assertEqual(repr(output), expected)
3235

36+
@patch('speech.shortcutKeys.shouldUseSpellingFunctionality', lambda: False)
37+
def test_simpleLetterKeyWithSpellingFunctionalityDisabled(self):
38+
"""A shortcut consisting in only one letter in the case where "Use spelling functionality" is disabled
39+
(see #15566).
40+
."""
41+
42+
expected = repr([
43+
'A',
44+
])
45+
output = _getKeyboardShortcutSpeech(
46+
keyboardShortcut='A',
47+
)
48+
self.assertEqual(repr(output), expected)
49+
50+
3351
def test_simpleSymbolKey(self):
3452
"""A shortcut consisting in only one symbol present in symbols.dic."""
3553

@@ -52,6 +70,7 @@ def test_simpleKeyName(self):
5270
)
5371
self.assertEqual(repr(output), expected)
5472

73+
@patch('speech.shortcutKeys.shouldUseSpellingFunctionality', lambda: True)
5574
def test_modifiersAndLetterKey(self):
5675
"""A shortcut consisting in modifiers and a letter key."""
5776

@@ -110,6 +129,7 @@ def test_modifierAndPlusKeyDescription(self):
110129
)
111130
self.assertEqual(repr(output), expected)
112131

132+
@patch('speech.shortcutKeys.shouldUseSpellingFunctionality', lambda: True)
113133
def test_sequentialShortcutCombiningSpacesAndCommas(self):
114134
"""A sequential shortcut found in ribbons (e.g. Word)."""
115135

@@ -136,6 +156,7 @@ def test_sequentialShortcutCombiningSpacesAndCommas(self):
136156

137157
class Test_getKeyboardShortcutsSpeech(unittest.TestCase):
138158

159+
@patch('speech.shortcutKeys.shouldUseSpellingFunctionality', lambda: True)
139160
def test_twoShortcutKeys(self):
140161
"""A shortcut key indication indicating two shortcut keys (a sequential one and a simultaneous one)
141162
as found in ribbons (e.g. Word).

0 commit comments

Comments
 (0)