Skip to content

Comments

Beta to master#17915

Merged
seanbudd merged 2 commits intomasterfrom
beta
Apr 8, 2025
Merged

Beta to master#17915
seanbudd merged 2 commits intomasterfrom
beta

Conversation

@seanbudd
Copy link
Member

@seanbudd seanbudd commented Apr 8, 2025

No description provided.

michaelDCurran and others added 2 commits April 8, 2025 12:07
…ate.py (#17909)

Summary of the issue:
The regenerate English user docs GitHub action is failing to run on beta as markdownTranslate.py moved from user_docs to source.

Description of user facing changes
None.

Description of development approach
Correct the path.
…cho mode is 'Only in edit controls'. (#17913)

Addresses #17670 for the classic Windows Calculator. Follow-up to PR #17905.

Summary of the issue:

Similar to the issue observed in the modern Windows Calculator, the classic Windows Calculator's display area does not consistently trigger character echo when NVDA's "Speak typed characters" setting is configured to 'Only in edit controls'.

Description of user facing changes

Users will now hear typed characters echoed when typing into the display field of the classic Windows Calculator, even when the "Speak typed characters" option is set to 'Only in edit controls'.

Description of development approach

This change mirrors the approach taken for the modern Calculator in PR #17905.
The event_typedCharacter method within the Display overlay class (which specifically targets the display controls of the classic Calculator in calc.py) is modified. When this event occurs and the original speakTypedCharacters mode is EDIT_CONTROLS, the mode is temporarily switched to ALWAYS before calling the parent class's event handler. The original mode is then restored in a finally block.
@seanbudd seanbudd requested a review from a team as a code owner April 8, 2025 06:25
@seanbudd seanbudd requested a review from SaschaCowley April 8, 2025 06:25
@seanbudd seanbudd merged commit a503ae3 into master Apr 8, 2025
5 of 7 checks passed
@github-actions github-actions bot added this to the 2025.2 milestone Apr 8, 2025
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.

3 participants