Skip to content

Commit

Permalink
fix #312083: tempofield in new score wizard not accessible
Browse files Browse the repository at this point in the history
Resolves: https://musescore.org/en/node/312083

A quirk in Qt prevents screen readers from recognizing UI elements
that are part of groups that were disabled in the UI file.
Fix is to enable the group in the UI file
but then disable it in the code.
  • Loading branch information
MarcSabatella authored and vpereverzev committed Nov 14, 2020
1 parent db33330 commit 9691f78
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions mscore/newwizard.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ TimesigWizard::TimesigWizard(QWidget* parent)
connect(tsCutTime, SIGNAL(toggled(bool)), SLOT(cutTimeToggled(bool)));
connect(tsFraction, SIGNAL(toggled(bool)), SLOT(fractionToggled(bool)));
pickupMeasure->setChecked(false); // checked in the UI file to enable screen reader on pickup duration controls
tempoGroup->setChecked(false);

tsCommonTime->setIcon(*icons[int(Icons::timesig_common_ICON)]);
tsCutTime->setIcon(*icons[int(Icons::timesig_allabreve_ICON)]);
Expand Down
2 changes: 1 addition & 1 deletion mscore/timesigwizard.ui
Original file line number Diff line number Diff line change
Expand Up @@ -427,7 +427,7 @@
<bool>true</bool>
</property>
<property name="checked">
<bool>false</bool>
<bool>true</bool>
</property>
<layout class="QHBoxLayout">
<item>
Expand Down

0 comments on commit 9691f78

Please sign in to comment.