Bug fix when opening General settings dialog while command line --lan…#14578
Merged
Conversation
…g=Windows is used
michaelDCurran
approved these changes
Jan 29, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Link to issue number:
Fixes #14407
Summary of the issue:
When launching NVDA with parameter
--lang=windows, an error occurs when openeing the General settings dialog.This error probably occurs when Windows language is not in the list of languages supported by NVDA (even if I have not been able to test such Windows localization). This error also occurs when Windows provides language+region code ('fr_FR') whereas NVDA supports 'fr' more generally.
Description of user facing changes
For Windows language not directly supported by NVDA, use the language code as a description instead of a full text description.
Description of development approach
I have choosen to replace full text description by the language code when NVDA does not support windows language+region. This is the simplest approach.
For fr_FR, I may have implemented a fallback to 'fr', but this would just throw other corner cases for Windows languages completely unknown to NVDA. I believe that displaying full text language names in a few more cases do not deserve a much more complex implementation. The first goal is to be able to open the settings dialog.
Testing strategy:
Manual tests:
Launch NVDA with various arguments and open the General settings dialogs:
--lang=Windows--lang=fr--lang=it--lang=pt_BR--langparameterKnown issues with pull request:
Language description of command line argument is not always a full description.
Change log entries:
Bug fixes
When NVDA is launched with --lang=Windows parameter, it is again to open General settings dialog. (#14407)Code Review Checklist: