App module for Winword to gather the code specific to MSWord#11824
Conversation
The code common to MS Word and other applications (such as MS Outlook) should not be stored in this module but in common object files such as NVDAObjects/window/winword.py. For now this appModule contains only toggleChangeTracking script since this feature is available in Word only.
Co-authored-by: Leonard de Ruijter <leonardder@users.noreply.github.com>
|
I have just edited the initial description, Change log entry section, to add a line in the Change for developers section. @feerrenrut this PR has been reviewed. Regarding French gesture.ini file, I will take care of it. |
|
For reference the fr gesture.ini file has been updated in SVN screenreadertranslation repo, revision 59653. |
Link to issue number:
Fixes #11196 (with adapted translation for the
gestures.inifile)Summary of the issue:
The NVDAObjects\window\winword.WordDocument class gathers some scripts that apply to word documents either in MS Word than in MS Outlook.
For Outlook, there is a specific app module defining among others a WordDocument subclass embedding the code specific to Outlook that does not apply to word.
However a subclass of NVDAObjects\window\winword.WordDocument applying to MS Word document only (i.e. not to Outlook one) is missing to embed code that applies to Word but not to Outlook. I have found two cases where it is missing.
Case 1: script toggleChangeTracking
Since change tracking is a Word only feature, the toggleChangeTracking script should not be called in Outlook. When using French MS Office and NVDA, we have control+shift+R being the shortcut for change tracking toggle in MS Word, but control+shift+R is the shortcut for "Reply all" in Outlook. This may cause unexpected errors (see #11196 for details).
Sometimes also one can hear a part of change tracking state reporting when pressing control+shift+R (reply all); this is however interrupted by new focused message window.
Case 2: script toggleAlignment
Toggle paragraph alignment commands exist both in Word and Outlook. However in French version they do not have exactly the same shortcuts (Office 2016):
In present version however, NVDA does not allow to assign separate shortcuts for Word or Outlook.
Description of how this pull request fixes the issue:
When
gestures.iniis translated accordingly, this will solve the case 1.Regarding case 2, NVDA's French
gestures.inifile may modified to associate scripts only where applicable (Word, Outlook or Word+Outlook), provided that all versions of Office have the same shortcuts. If this is not the case, the modifications may still be done at user level in the personalgestures.inifile. I will investigate this point with other French translators and users.Testing performed:
Just for test purpose, I have modified the French gesture.ini as follows:
Or download the modified French
gestures.inifilegestures.ini.txt.
The changes in the modified
gestures.inifile are the following:[appModules.winword.WinwordWordDocument].[NVDAObjects.window.winword.WordDocument]toggleAlignmentmapping tokb:control+shift+gin this section rather than in[NVDAObjects.window.winword.WordDocument]With this file, I have checked that:
Known issues with pull request:
gesture.inineed to be modified accordingly; I will take care of itChange log entry:
Bug fixes:
In Outlook French version, the shortcut for 'Reply all' (control+shift+R) should be working again. (#11196)Changes for developers:
It is now possible to define separate gestures for Outlook and Word document. (#11196)