Skip to content

Add a braille setting for Chinese word segmentation - #20642

Draft
cary-rowen wants to merge 1 commit into
nvaccess:masterfrom
cary-rowen:add-braille-chinese-word-segmentation-toggle
Draft

Add a braille setting for Chinese word segmentation#20642
cary-rowen wants to merge 1 commit into
nvaccess:masterfrom
cary-rowen:add-braille-chinese-word-segmentation-toggle

Conversation

@cary-rowen

@cary-rowen cary-rowen commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Link to issue number:

Fixes #20471

Summary of the issue:

Chinese word segmentation is always used with Chinese braille output tables.
Some users need to disable this behavior and let the selected braille table translate the original text directly.

Description of user facing changes:

A "Use Chinese word segmentation" checkbox has been added to the Braille settings category.

It is enabled by default to preserve the existing behavior and only affects Chinese braille output tables.
Context-sensitive help and User Guide documentation are included.

Description of developer facing changes:

There are no public API changes.

Description of development approach:

The change follows the conventions of existing Braille settings and preserves the current behavior by default.

Testing strategy:

Manual testing should cover the following:

  • With a Chinese braille output table, confirm that word segmentation can be enabled and disabled.
  • Confirm that changing the setting updates the braille output as expected.
  • Confirm that the setting has no effect with non-Chinese braille output tables.
  • Press F1 on the setting and confirm that the corresponding User Guide section opens.

Known issues with pull request:

None known.

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.

@cary-rowen

Copy link
Copy Markdown
Contributor Author

HI cc @keyang556
Would you mind test this PR build?

@keyang556

Copy link
Copy Markdown
Contributor

OK, I'll test it.

@keyang556

Copy link
Copy Markdown
Contributor

Fixed, thank you.

@cary-rowen
cary-rowen marked this pull request as ready for review August 10, 2026 12:44
@cary-rowen
cary-rowen requested review from a team as code owners August 10, 2026 12:44
@cary-rowen

Copy link
Copy Markdown
Contributor Author

Thanks @keyang556

@keyang556

keyang556 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

I don't know, if possible, off by default in zhtw table(or nvda language), but on by default in zhcn table?

Qchristensen
Qchristensen previously approved these changes Aug 12, 2026

@Qchristensen Qchristensen 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.

User guide reads well

@seanbudd
seanbudd marked this pull request as draft August 13, 2026 00:20
@seanbudd
seanbudd requested a review from Qchristensen August 13, 2026 00:20
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.

Add an option to disable Chinese word separators in braille output

3 participants