Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Beta to master #16597

Merged
merged 2 commits into from
May 23, 2024
Merged

Beta to master #16597

merged 2 commits into from
May 23, 2024

Conversation

seanbudd
Copy link
Member

No description provided.

mltony and others added 2 commits May 24, 2024 09:43
Fixes #16546

Summary of the issue:
Style navigation is too slow in MSWord even with UIA enabled.

Description of user facing changes
Message "not supported in this document" is spoken when trying style navigation in MSWord.

Description of development approach
I did a bit of refactoring. Instead of trying to catch all the conditions in browseMode.BrowseModeDocumentTreeInterceptor, I overrode _iterTextStyle() method in corresponding child subclasses.
Related to #16345.

Summary of the issue:
Multiple dots are reported as "padding dots" in situations where these dots have no padding function. "padding" is too restrictive and is also more difficult to understand; by the way, some translators have actually translated "padding dots" to "multiple dots" in their translations.

Description of user facing changes
Multiple dots (4 or more) will now be reported with the more neutral "multiple dots" instead of "padding dots" when the symbol level is high enough.

Description of development approach
Changed both the symbol name and what is reported in symbol file.
@seanbudd seanbudd requested a review from a team as a code owner May 23, 2024 23:52
@seanbudd seanbudd requested a review from gerald-hartig May 23, 2024 23:52
@seanbudd seanbudd merged commit 4c53f4a into master May 23, 2024
1 of 3 checks passed
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