nvda icon indicating copy to clipboard operation
nvda copied to clipboard

Split up scriptHandler.findScript

Open seanbudd opened this issue 1 year ago • 0 comments

Link to issue number:

None

Summary of the issue:

scriptHandler.findScript is considered too complex. Therefore it should be broken up into smaller functions.

Description of user facing changes

None

Description of development approach

Added typing to related objects. Split up scriptHandler.findScript so it is more modular.

Testing strategy:

Tested various input gestures.

Known issues with pull request:

None

Change log entries:

None

Code Review Checklist:

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

seanbudd avatar Aug 05 '22 06:08 seanbudd