Cannot navigate to repeat text (measure elements)

• Sep 5, 2019 - 22:59
Reported version
3.2
Priority
P1 - High
Type
Functional
Frequency
Once
Severity
S4 - Minor
Reproducibility
Always
Status
closed
Regression
No
Workaround
No
Project

Currently, there is no way to navigate to Coda signs etc. My recent PR https://github.com/musescore/MuseScore/pull/5270 adds screenreader support for these and other symbols, but since these are added to the score as measure elements, the current next/previous-element commands skip them. Adding that support should be similar to what I did to add support for frames in https://github.com/musescore/MuseScore/pull/5270/commits/5bf46eae2f28a2…, but we'd need more code to traverse multiple elements on a measure.

Relates to #277496: [EPIC] Accessibility issues/suggestions


Comments

Status PR created fixed

Fixed in branch master, commit 94d989429e

_fix #294126: cannot navigate to measure elements

The navigation code for next-element and previous-elements ignores measure elements
(elements in the el() list for the measure).
This change adds handling for this
by checking for measure elements before moving to the next or previous measure
in Segment::nextElement() and Segment::previousElement(),
There is corresponding code in Measure::nextElementStaff()
and in Measure::previousElementStaff()
to iterate through multiple elements if present._

Fix version
3.3.0