Skip to content

Check existence of braille handler display before fetching gestureMap#15051

Merged
seanbudd merged 1 commit into
masterfrom
fix-14750
Jun 26, 2023
Merged

Check existence of braille handler display before fetching gestureMap#15051
seanbudd merged 1 commit into
masterfrom
fix-14750

Conversation

@seanbudd

@seanbudd seanbudd commented Jun 23, 2023

Copy link
Copy Markdown
Member

Link to issue number:

Fixes #14750

Summary of the issue:

If a braille display is initialized, but then disconnected, the input gestures dialog cannot be opened.

Description of user facing changes

Description of development approach

As per handling in scriptHandler, we must check the existence of braille handler display before fetching gestureMap

globalMap = braille.handler.display.gestureMap if braille.handler and braille.handler.display else None

Testing strategy:

Known issues with pull request:

None

Change log entries:

N/A fixes regression?

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.

@seanbudd seanbudd requested a review from a team as a code owner June 23, 2023 05:30
@seanbudd seanbudd requested a review from michaelDCurran June 23, 2023 05:30
@seanbudd

Copy link
Copy Markdown
Member Author

@seanbudd seanbudd merged commit 23b4992 into master Jun 26, 2023
@seanbudd seanbudd deleted the fix-14750 branch June 26, 2023 00:20
@nvaccessAuto nvaccessAuto added this to the 2023.2 milestone Jun 26, 2023
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.

Input gestures dialog fails to start

3 participants