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

Accessibility: Improve speech output for notes and common elements #12340

Merged
merged 1 commit into from
Jul 21, 2022

Conversation

shoogle
Copy link
Contributor

@shoogle shoogle commented Jul 11, 2022

Screen readers will now announce:

  • Accidentals in note names (e.g. "A♯", "B♭", etc.).

  • [Fix MU#312625] Notehead group type (e.g. "cross", "triangle", etc.).

  • [Fix MU#312628] When notes or rests use cross-staff notation.

  • [Fix MU#312627] When elements are invisible.

  • [Fix MU#307442] Tuplet length for custom tuplets (more than 9 notes).

Replaces PR #8877.

@shoogle shoogle force-pushed the accessibility-element-info branch from e81c0bf to 533f291 Compare July 11, 2022 13:46
@shoogle shoogle added the strings Affects translatable strings label Jul 11, 2022
@shoogle shoogle force-pushed the accessibility-element-info branch from 533f291 to 1ae3de3 Compare July 11, 2022 15:31
@shoogle
Copy link
Contributor Author

shoogle commented Jul 11, 2022

Tested working with NVDA, Narrator, VoiceOver, Orca.

For testing purposes, use Ctrl+Shift+Up or Ctrl+Shift+Down to move some notes across staves, then make the note invisible or change the notehead shape via the Properties panel.

image

Create a custom tuplet with 10 or more notes via:
image image
image

And check speech output for the above.

@shoogle shoogle force-pushed the accessibility-element-info branch from e2cc066 to 93067d2 Compare July 12, 2022 02:14
Screen readers will now announce:

- [Fix #312625] Notehead group type (e.g. "cross", "triangle", etc.).

- [Fix #312628] When notes or rests use cross-staff notation.

- [Fix #312627] When elements are invisible.

- [Fix #307442] Tuplet length for custom tuplets (more than 9 notes).

Also speeds up speech output by stripping some punctuation from element
names to prevent unnecessary pauses.
@Eism Eism merged commit 222a8b6 into musescore:master Jul 21, 2022
@shoogle shoogle deleted the accessibility-element-info branch July 28, 2022 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
strings Affects translatable strings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants