Skip to content

Fix SAPI5 - #18802

Merged
seanbudd merged 1 commit into
nvaccess:betafrom
gexgd0419:sapi5-fix
Aug 26, 2025
Merged

Fix SAPI5#18802
seanbudd merged 1 commit into
nvaccess:betafrom
gexgd0419:sapi5-fix

Conversation

@gexgd0419

@gexgd0419 gexgd0419 commented Aug 25, 2025

Copy link
Copy Markdown
Contributor

Link to issue number:

Fixes #18797.

Summary of the issue:

NVDA is silent after executing certain gestures.

Description of user facing changes:

The problem mentioned above should be fixed.

Description of developer facing changes:

None.

Description of development approach:

Reset the _isCancelled flag to False before speaking. Otherwise, calling cancel before speak may silence the later speak.

Testing strategy:

Tested manually.

Known issues with pull request:

None.

Code Review Checklist:

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

@seanbudd seanbudd added this to the 2025.3 milestone Aug 25, 2025
@gexgd0419
gexgd0419 marked this pull request as ready for review August 26, 2025 03:07
@gexgd0419
gexgd0419 requested a review from a team as a code owner August 26, 2025 03:07
@gexgd0419
gexgd0419 requested a review from SaschaCowley August 26, 2025 03:07
@seanbudd
seanbudd merged commit c7e5aae into nvaccess:beta Aug 26, 2025
30 checks passed
@gexgd0419
gexgd0419 deleted the sapi5-fix branch August 26, 2025 04:44
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.

2 participants