add option to disable beeping on passwords fields

add basic validation on tone pitch, length and volume fields. User can't save non digit characters or negative numbers.
update english, spanish and french readme.
update translation strings template
update spanish and french languages strings.

Author: davidacm

addon/doc/es/readme.md

@@ -12,7 +12,7 @@ Este paquete es distribuido bajo los términos de la GNU General Public License,
 * Pitar para caracteres escritos cuando la tecla shift es presionada: con esta característica NVDA pitará si escribes un carácter con la tecla shift presionada.
 * Pitar cuando cambia el estado de las teclas interruptores: con esta característica, NVDA producirá un tono alto cuando el interructor se activa, y uno bajo si se desactiva.
 * Anunciar el cambio de estado de las teclas interruptores: Solo para cuando "Pitar cuando cambia el estado de las teclas" está habilitado. Puedes habilitar o deshabilitar el anunciado de las teclas interruptores cuando estas cambian su estado.
-* pitar para caracteres específicos: en las opciones avanzadas, puedes configurar NVDA para que emita un pitido con los caracteres que desees.
+* Pitar para caracteres específicos: en las opciones avanzadas, puedes configurar NVDA para que emita un pitido con los caracteres que desees.
 
 ## Requisitos
   Necesitarás NVDA 2018.2 o posterior.
@@ -23,3 +23,13 @@ Este paquete es distribuido bajo los términos de la GNU General Public License,
 ## Uso
   Para desactivar o activar características, ve  a las configuraciones de NVDA y selecciona la categoría beep keyboard. En esa categoría podrás habilitar o deshabilitar cualquier característica soportada por este complemento.  
   "Pitar para mayúsculas cuando el bloqueo mayúsculas está activado" está habilitado por defecto.
+  Si necesitas ajustes adicionales, abra el cuadro de diálogo de las opciones avanzadas que contiene las siguientes opciones:
+
+* Ignorar caracteres escritos con la tecla shift presionada: todos los caracteres escritos aquí serán ignorados al emitir el pitido cuando la tecla shift sea presionada. Se permiten secuencias de escape, EJ. "\t" para tab, "\r" para la tecla enter.
+* Pitar siempre para los siguientes caracteres: ingrese aquí todos los caracteres escritos para los que quieras que NVDA emita un pitido. Se permiten secuencias  de escape, EJ. "\t" para tab, "\r" para la tecla enter.
+* Seleccionar un tono para su configuración: puedes configurar parámetros para todos los tonos. Selecciona uno aquí, y ajuste los parámetros en los siguientes cuadros de texto. Al cambiar de opción, NVDA emitirá el pitido del tono seleccionado con los parámetros configurados en ese momento.
+* Tono: tono para el tono seleccionado actualmente.
+* Duración: duración para el tono seleccionado actualmente.
+* Volumen: volumen para el tono seleccionado actualmente.
+* Probar el tono: este botón te permite reproducir un tono de prueba con los parámetros establecidos actualmente.
+* Presione el botón Aceptar para guardar las configuraciones o cancelar para descartarlas.

addon/doc/fr/readme.md

@@ -10,8 +10,9 @@ Ce paquet est distribué sous les termes de la licence publique générale GNU,
 
 * Bip pour les majuscules quand le verrouillage majuscule est activé: si cette fonctionnalité est activée, NVDA émettra un bip lorsque vous tapez une lettre majuscule et que le verrouillage majuscule est activé. Ne faites plus d'erreurs de majuscules !
 * Bip pour les caractères saisis lorsque vous appuyez sur la touche maj: avec cette fonctionnalité, NVDA émettra un bip si vous tapez un caractère avec la touche maj enfoncée.
-* Beep quand l'état des touches de l'interrupteur à bascule change: avec cette fonctionnalité, NVDA émettra un bip plus aigu si l'interrupteur à bascule est activé et une tonalité plus grave s'il est désactivé.
-* Annoncer le changement de l'état des  touches de l'interrupteur à bascule: uniquement lorsque l'option "Beep pendant le changement de l'état des touches de l'interrupteur à bascule" est activé. Vous pouvez activer ou désactiver l'annonce des  touches de l'interrupteur à bascule lorsqu'elles changent d'état.
+* Bip quand l'état des touches de l'interrupteur à bascule change: avec cette fonctionnalité, NVDA émettra un bip plus aigu si l'interrupteur à bascule est activé et une tonalité plus grave s'il est désactivé.
+* Annoncer le changement de l'état des  touches de l'interrupteur à bascule: uniquement lorsque l'option "Bip pendant le changement de l'état des touches de l'interrupteur à bascule" est activé. Vous pouvez activer ou désactiver l'annonce des  touches de l'interrupteur à bascule lorsqu'elles changent d'état.
+* Bip pour les caractères spécifiés: dans les paramètres avancés, vous pouvez configurer NVDA pour émettre un bip avec les caractères souhaités.
 
 ## Exigences
   Vous avez besoin de NVDA 2018.2 ou une version ultérieure.
@@ -22,3 +23,13 @@ Ce paquet est distribué sous les termes de la licence publique générale GNU,
 ## Utilisation
   Pour activer ou désactiver des fonctionnalités, accédez aux paramètres NVDA et sélectionnez la catégorie beep keyboard. Dans cette catégorie, vous pouvez configurer toutes les fonctionnalités prises en charge par l'extension.  
   "Bip pour les majuscules quand le verrouillage majuscule est activé" est activé par défaut.
+  Si vous avez besoin de réglages supplémentaires, ouvrez la boîte de dialogue des paramètres avancés contenant les paramètres suivants:
+
+* Ignorés les caractères saisis avec la touche maj enfoncée: tous les caractères saisis ici seront ignorés lors de l'émission du bip quand on appuie sur  la touche maj. Les séquences d'échappement sont autorisées, par exemple "\t" pour tab, "\r" pour la touche Entrée.
+* Bip toujours pour les caractères suivants: insérez ici tous les caractères pour lesquels vous souhaitez que NVDA émette un bip. Les séquences d'échappement sont autorisées, par exemple "\t" pour tab, "\r" pour la touche Entrée.
+* Choisir la tonalité à configurer: vous pouvez définir des paramètres pour toutes les tonalités. Sélectionnez-en une ici et ajustez les paramètres dans les zones de texte suivantes. Lors du changement des paramètres NVDA émettra  le bip de la tonalité sélectionnée avec les paramètres configurés à ce moment.
+* Hauteur: hauteur pour la tonalité actuellement sélectionnée.
+* Durée: durée pour la tonalité actuellement sélectionnée.
+* Volume: volume pour la tonalité actuellement sélectionnée.
+* Tester la tonalité: ce bouton vous permet de jouer une tonalité de test avec les paramètres actuellement définis.
+* Appuyer sur le bouton OK pour sauvegarder la configuration ou sur Annuler pour les écarter.

addon/globalPlugins/beepKeyboard.py

@@ -5,27 +5,28 @@
 # Released under GPL 2
 #globalPlugins/beepKeyboard.py
 
-import codecs, config, globalPluginHandler, gui, keyboardHandler, tones, winUser, wx, addonHandler
+import api, codecs, config, globalPluginHandler, gui, keyboardHandler, tones, ui, winUser, wx, addonHandler
 addonHandler.initTranslation()
 
 confspec = {
 	"beepUpperWithCapsLock": "boolean(default=True)",
 	"beepCharacterWithShift": "boolean(default=False)",
 	"beepToggleKeyChanges": "boolean(default=False)",
 	"announceToggleStatus": "boolean(default=True)",
+	"disableBeepingOnPasswordFields": "boolean(default=True)",
 	"ignoredCharactersForShift": "string(default='\t\b\r ')",
 	"beepForCharacters": "string(default='')",
 	"shiftedCharactersTone": "int_list(default=list(6000,10,25))",
 	"customCharactersTone": "int_list(default=list(6000,10,25))",
 	"capsLockUpperTone": "int_list(default=list(3000,40,50))",
 	"toggleOffTone": "int_list(default=list(1000,40,50))",
-	"toggleOnTone": "int_list(default=list(2000, 40, 50))",
+	"toggleOnTone": "int_list(default=list(2000, 40, 50))"
 }
 config.conf.spec["beepKeyboard"] = confspec
 
 def beep(l):
 	""" it receives a list with three arguments to beep: [pitch, length, volume]"""
-	tones.beep(*l, right=l[-1])
+	if not (config.conf['beepKeyboard']['disableBeepingOnPasswordFields'] and api.getFocusObject().isProtected): tones.beep(*l, right=l[-1])
 
 #saves the original _reportToggleKey function
 origReportToggleKey = keyboardHandler.KeyboardInputGesture._reportToggleKey
@@ -34,7 +35,7 @@ def _reportToggleKey(self):
 	if winUser.getKeyState(self.vkCode) & 1:
 		beep(config.conf['beepKeyboard']['toggleOnTone'])
 	else: beep(config.conf['beepKeyboard']['toggleOffTone'])
-	if config.conf['beepKeyboard']['announceToggleStatus']: origReportToggleKey(self)
+	if config.conf['beepKeyboard']['announceToggleStatus'] or (config.conf['beepKeyboard']['disableBeepingOnPasswordFields'] and api.getFocusObject().isProtected): origReportToggleKey(self)
 
 class BeepKeyboardSettingsPanel(gui.SettingsPanel):
 	# Translators: This is the label for the beepKeyboard  settings category in NVDA Settings screen.
@@ -54,6 +55,9 @@ def makeSettings(self, settingsSizer):
 		# Translators: label for a checkbox option in the settings panel.
 		self.announceToggleStatus = sHelper.addItem(wx.CheckBox(self, label=_("&Announce toggle keys changes (if Beep for toggle keys changes is disabled NVDA will have the original behavior)")))
 		self.announceToggleStatus.SetValue(config.conf['beepKeyboard']['announceToggleStatus'])
+		# Translators: label for a checkbox option in the settings panel.
+		self.disableBeepingOnPasswordFields = sHelper.addItem(wx.CheckBox(self, label=_("&disable beeping on password fields")))
+		self.disableBeepingOnPasswordFields.SetValue(config.conf['beepKeyboard']['disableBeepingOnPasswordFields'])
 
 		# Translators: label for a button to open advanced settings dialog in the settings panel.
 		advancedButton = sHelper.addItem (wx.Button (self, label = _("&Open advanced options")))
@@ -68,11 +72,11 @@ def onSave(self):
 		config.conf['beepKeyboard']['beepCharacterWithShift'] = self.beepCharacterWithShift.GetValue()
 		config.conf['beepKeyboard']['beepToggleKeyChanges'] = self.beepToggleKeyChanges.GetValue()
 		config.conf['beepKeyboard']['announceToggleStatus'] = self.announceToggleStatus.GetValue()
+		config.conf['beepKeyboard']['disableBeepingOnPasswordFields'] = self.disableBeepingOnPasswordFields.GetValue()
 		if hasattr(config, "post_configProfileSwitch"):
 			config.post_configProfileSwitch.notify()
 		else: config.configProfileSwitched.notify()
 
-
 class AdvancedBeepKeyboardSettingsDialog(gui.SettingsDialog):
 	# Translators: This is the label for the beep keyboard advanced settings dialog
 	title = _("Advanced settings - Keyboard beep")
@@ -85,8 +89,7 @@ def makeSettings(self, settingsSizer):
 		self.ignoredCharactersForShift.SetValue(co['ignoredCharactersForShift'])
 		# Translators: label for an edit text control option in the advanced settings dialog.
 		self.beepForCharacters  = sHelper.addLabeledControl(_("Beep &always for the following characters"), wx.TextCtrl)
-		self.beepForCharacters.SetValue(co['beepForCharacters'])
-		
+		self.beepForCharacters.SetValue(co['beepForCharacters'])		
 		self.tonesParameters = [co['shiftedCharactersTone'], co['customCharactersTone'], co['capsLockUpperTone'], co['toggleOffTone'], co['toggleOnTone']]
 		# Translators: label for a combo box control in the advanced settings dialog.
 		self.toneOptionsList = sHelper.addLabeledControl(_("&Select tone to configure"), wx.Choice, choices=[
@@ -121,14 +124,26 @@ def postInit(self):
 		self.ignoredCharactersForShift.SetFocus()
 
 	def updateCurrentToneValues(self):
-		self.tonesParameters[self.curSelection] = [int(self.tonePitch.GetValue()), int(self.toneLength.GetValue()), int(self.toneVolume.GetValue())]
+		try:
+			l = [int(self.tonePitch.GetValue()), int(self.toneLength.GetValue()), int(self.toneVolume.GetValue())]
+			if True in [k<0 for k in l]: raise ValueError
+			self.tonesParameters[self.curSelection] = l
+		except ValueError:
+			tones.beep(100,160)
+			# Translators: an error message sent to ve spoken by NVDA.
+			ui.message(_("You entered a wrong value. Please correct it or discard changes."))
+			raise ValueError
 
 	def onTest(self, evt):
 		self.updateCurrentToneValues()
 		beep(self.tonesParameters[self.curSelection])
 
 	def onToneOptionChange(self, evt):
-		self.updateCurrentToneValues()
+		try:
+			self.updateCurrentToneValues()
+		except ValueError:
+			self.toneOptionsList.SetSelection(self.curSelection)
+			return
 		self.curSelection = self.toneOptionsList.GetCurrentSelection()
 		co = self.tonesParameters[self.curSelection][0:3]
 		beep(co)
@@ -137,14 +152,10 @@ def onToneOptionChange(self, evt):
 		self.toneVolume.SetValue(str(co[2]))
 
 	def onOk(self, evt):
+		self.updateCurrentToneValues()
 		config.conf['beepKeyboard']['ignoredCharactersForShift'] = self.ignoredCharactersForShift.GetValue()
 		config.conf['beepKeyboard']['beepForCharacters'] = self.beepForCharacters.GetValue()
-		self.updateCurrentToneValues()
-		config.conf['beepKeyboard']['shiftedCharactersTone'] = self.tonesParameters[0]
-		config.conf['beepKeyboard']['customCharactersTone'] = self.tonesParameters[1]
-		config.conf['beepKeyboard']['capsLockUpperTone'] = self.tonesParameters[2]
-		config.conf['beepKeyboard']['toggleOffTone'] = self.tonesParameters[3]
-		config.conf['beepKeyboard']['toggleOnTone'] = self.tonesParameters[4]
+		config.conf['beepKeyboard']['shiftedCharactersTone'], config.conf['beepKeyboard']['customCharactersTone'], config.conf['beepKeyboard']['capsLockUpperTone'], config.conf['beepKeyboard']['toggleOffTone'], config.conf['beepKeyboard']['toggleOnTone'] = self.tonesParameters
 		if hasattr(config, "post_configProfileSwitch"):
 			config.post_configProfileSwitch.notify()
 		else: config.configProfileSwitched.notify()
@@ -167,8 +178,7 @@ def event_typedCharacter(self, obj, nextHandler, ch):
 			beep(config.conf['beepKeyboard']['capsLockUpperTone'])
 		elif config.conf['beepKeyboard']['beepCharacterWithShift'] and not winUser.getKeyState(winUser.VK_CONTROL) &32768 and winUser.getKeyState(winUser.VK_SHIFT) &32768 and ch not in self.ignoredCharactersForShift and not (config.conf["keyboard"]["beepForLowercaseWithCapslock"] and ch.islower() and winUser.getKeyState(winUser.VK_CAPITAL)&1):
 			beep(config.conf['beepKeyboard']['shiftedCharactersTone'])
-		elif ch in self.beepForCharacters:
-			beep(config.conf['beepKeyboard']['customCharactersTone'])
+		elif ch in self.beepForCharacters: beep(config.conf['beepKeyboard']['customCharactersTone'])
 
 	def setExternalReportToggleStatus(self, flag):
 		if flag: