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
[MU3] Fix typo in a the name of a style setting #7711
Conversation
I'm slightly puzzled that 3.x needs these changes to vtests for them to pass, but for master it does not? |
@@ -434,7 +434,7 @@ static const StyleType styleTypes[] { | |||
{ Sid::mmRestNumberPos, "mmRestNumberPos", Spatium(-1.5) }, | |||
{ Sid::hideEmptyStaves, "hideEmptyStaves", QVariant(false) }, | |||
{ Sid::dontHideStavesInFirstSystem, | |||
"dontHidStavesInFirstSystm", QVariant(true) }, | |||
"dontHideStavesInFirstSystem", QVariant(true) }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this name change cause compatibility issues with existing scores?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, that's what the 2nd chunk of this change is about.
It may (read: will) cause downrev issues though, like when a score created woith this PR merged is read in a version before that, but that is for lyricsDashMaxLegth
vs. lyricsDashMaxLength
too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I missed that one.
This reverts commit 89e9471, PR musescore#7711. Causes issues when reading scores containing "dontHideStavesInFirstSystem" with earlier 3.x releases.
This reverts commit 89e9471, PR musescore#7711. Causes issues when reading scores containing "dontHideStavesInFirstSystem" with earlier 3.x releases.
For master see #7712