Skip to content

Minor consistency fixes in MathCAT preferences#19440

Merged
seanbudd merged 1 commit into
nvaccess:betafrom
codeofdusk:fixup-mathcat-20260112
Jan 16, 2026
Merged

Minor consistency fixes in MathCAT preferences#19440
seanbudd merged 1 commit into
nvaccess:betafrom
codeofdusk:fixup-mathcat-20260112

Conversation

@codeofdusk

Copy link
Copy Markdown
Contributor

Minor follow-up of #19227.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves the consistency and clarity of MathCAT preference UI strings as a follow-up to #19227, which addressed unclear MathCAT options.

Changes:

  • Changed "Auto" to "Automatic" for the decimal separator option
  • Reordered "Use Voice's Language (Auto)" to "Automatic (Use Voice's Language)" to match NVDA's standard UI pattern

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
source/mathPres/MathCAT/preferences.py Updated decimal separator option label from "Auto" to "Automatic" for consistency
source/mathPres/MathCAT/localization.py Reordered language selection string to "Automatic (Use Voice's Language)" to follow NVDA's standard pattern where "Automatic" comes first, and updated corresponding docstring

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@seanbudd seanbudd added this to the 2026.1 milestone Jan 15, 2026

@seanbudd seanbudd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @codeofdusk

@seanbudd seanbudd merged commit 84e14ee into nvaccess:beta Jan 16, 2026
42 of 45 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