Skip to content

Bug fix when opening General settings dialog while command line --lan…#14578

Merged
michaelDCurran merged 3 commits into
nvaccess:masterfrom
CyrilleB79:langCmdLine
Jan 29, 2023
Merged

Bug fix when opening General settings dialog while command line --lan…#14578
michaelDCurran merged 3 commits into
nvaccess:masterfrom
CyrilleB79:langCmdLine

Conversation

@CyrilleB79

Copy link
Copy Markdown
Contributor

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
  • no --lang parameter

Known 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:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • Security precautions taken.

@CyrilleB79 CyrilleB79 marked this pull request as ready for review January 23, 2023 13:07
@CyrilleB79 CyrilleB79 requested a review from a team as a code owner January 23, 2023 13:07
@CyrilleB79 CyrilleB79 requested a review from seanbudd January 23, 2023 13:07
@michaelDCurran michaelDCurran merged commit c3d68a3 into nvaccess:master Jan 29, 2023
@nvaccessAuto nvaccessAuto added this to the 2023.1 milestone Jan 29, 2023
@CyrilleB79 CyrilleB79 deleted the langCmdLine branch January 30, 2023 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error when opening NVDA general settings

3 participants