Skip to content

Refine Unicode normalization paragraph in User Guide#19898

Merged
seanbudd merged 1 commit intonvaccess:masterfrom
CyrilleB79:UGNorm
Apr 7, 2026
Merged

Refine Unicode normalization paragraph in User Guide#19898
seanbudd merged 1 commit intonvaccess:masterfrom
CyrilleB79:UGNorm

Conversation

@CyrilleB79
Copy link
Copy Markdown
Contributor

Link to issue number:

Fix-up of #19608

Summary of the issue:

With the introduction of #19608, the logic of the Unicode normalization paragraph in the User Guide has been broken. Namely, the information regarding additional (non-NFKC) normalization has been added as a bullet in the list of NVKC benefits.

Description of user facing changes:

Unicode normalization paragraph modified / updated.

Description of developer facing changes:

N/A

Description of development approach:

  • Separate description of N FKC benefits and additional NVDA normalization.
  • Also other small fixes, e.g. upper/lowercase, etc.

Testing strategy:

Check generated User Guide

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.

@CyrilleB79
Copy link
Copy Markdown
Contributor Author

Cc @bramd

@CyrilleB79 CyrilleB79 marked this pull request as ready for review April 3, 2026 17:57
@CyrilleB79 CyrilleB79 requested a review from a team as a code owner April 3, 2026 17:57
@CyrilleB79 CyrilleB79 requested a review from Qchristensen April 3, 2026 17:57
Copy link
Copy Markdown
Member

@Qchristensen Qchristensen left a comment

Choose a reason for hiding this comment

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

Reads well

@seanbudd seanbudd merged commit 0c9dfd1 into nvaccess:master Apr 7, 2026
41 checks passed
@github-actions github-actions bot added this to the 2026.2 milestone Apr 7, 2026
@bramd
Copy link
Copy Markdown
Contributor

bramd commented Apr 7, 2026

@CyrilleB79 Thanks for improving this

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.

4 participants