Skip to content

Add Unicode braille support#14548

Merged
seanbudd merged 8 commits intomasterfrom
addUnicodeBrailleSupport
Feb 14, 2023
Merged

Add Unicode braille support#14548
seanbudd merged 8 commits intomasterfrom
addUnicodeBrailleSupport

Conversation

@seanbudd
Copy link
Copy Markdown
Member

@seanbudd seanbudd commented Jan 16, 2023

Link to issue number:

Fixes #13778
Supersedes #13777

Summary of the issue:

If Unicode braille occurs within text — for instance, ⠐⠣⠃⠗⠇⠐⠜ says "(braille)" in UEB — NVDA currently just skips over the braille entirely. This is because it is not included in symbols.dic

Description of user facing changes

Unicode braille symbols are now announced.

Description of development approach

The sequence method attempted in #13777 fails in some scenarios as outlined in #13777 (comment)

Testing strategy:

Test reading braille symbols in symbol file

Known issues with pull request:

None

Change log entries:

New features:
Added pronunciation of Unicode braille symbols such as "⠐⠣⠃⠗⠇⠐⠜"

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • Security precautions taken.

saizai and others added 5 commits June 8, 2022 18:01
If Unicode braille occurs within text — for instance, ⠐⠣⠃⠗⠇⠐⠜ says "(braille)" in UEB — NVDA currently just skips over the braille entirely. This makes it at least pronounce the braille cells.

Because this is pronunciation for the Unicode characters themselves, it is braille-code-agnostic and just pronounces individual braille characters in the usual "dot 3 5 7" way. If the input table is known, this default should get overridden, but it's at least a fallback, and allows one to read text that's _about_ braille.

I've left level & preserve in defaults.
@seanbudd seanbudd marked this pull request as ready for review January 16, 2023 05:23
@seanbudd seanbudd requested a review from a team as a code owner January 16, 2023 05:23
@AppVeyorBot
Copy link
Copy Markdown

See test results for failed build of commit 340f6b6f38

Copy link
Copy Markdown
Member

@michaelDCurran michaelDCurran left a comment

Choose a reason for hiding this comment

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

It would be preferable to speak it as "braille 1 2 3" rather than "1 2 3 braille", as this is how eSpeak is already speaking it, and alerts the user that they are listening to braille before presenting random numbers.

@seanbudd
Copy link
Copy Markdown
Member Author

@michaelDCurran I've re-ordered it now

@seanbudd seanbudd merged commit ad69680 into master Feb 14, 2023
@seanbudd seanbudd deleted the addUnicodeBrailleSupport branch February 14, 2023 01:55
@nvaccessAuto nvaccessAuto added this to the 2023.2 milestone Feb 14, 2023
seanbudd pushed a commit that referenced this pull request Mar 23, 2023
NVDA doesn't announce the symbol that macs use to represent the mac "Option" key ("⌥") (see Mac keyboard shortcuts documentation).

Description of user facing changes
Adds a symbol description for this symbol ("mac Option key" announced at "none" level and up, similar to the existing entry for "mac Command key" at the "none" level for symbol "⌘").

Description of development approach
Followed the "Defining Symbol Information" developer documentation, using the existing "mac Command key" entry and similar recent PR #14548 as examples

Testing strategy:
Didn't attempt to adjust any automated tests (assumed they were already present generally for symbol pronunciation). Manually verified reading by:

runnvda.bat from this branch
Browse to Mac keyboard shortcuts docs
Read through the content under the "Mac keyboard shortcuts" H1, particularly the list item with content "Option (or Alt) ⌥"
Verify that the symbol is read out as "mac Option key" as expected
Change NVDA's "Settings > Speech > Punctuation/symbol level" to "none" and repeat above steps
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 pronunciation for raw Unicode braille cells

5 participants