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
Fix #274521: fix concert pitch clef issues #3824
Conversation
share/instruments/instruments.xml
Outdated
@@ -1066,10 +1066,13 @@ | |||
<shortName>Hph.</shortName> | |||
<description>Heckelphone</description> | |||
<musicXMLid>wind.reed.heckelphone</musicXMLid> | |||
<clef>G8vb</clef> | |||
<clef>G</clef> |
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.
<clef>
should be <transposingClef>
, and <concertClef>
should appear immediately afterwards. See piccolo for example.
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.
My bad, I'll change that and update the branch
share/instruments/instruments.xml
Outdated
@@ -2044,6 +2047,7 @@ | |||
<pPitchRange>22-60</pPitchRange> | |||
<transposeDiatonic>-7</transposeDiatonic> | |||
<transposeChromatic>-12</transposeChromatic> | |||
<concertClef>F8vb</concertClef> |
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.
Same problem here as above.
This looks good to me. Thanks for applying my suggestions. Now you just need to squash these commits into one single commit. Squashing commits
You can use the easy method. The advanced method is more complicated and is only useful for things that the easy method can't handle. I only mention the advanced method because for some reason it is usually given as the method for squashing commits, even though the easy method works fine for this. |
2515b93
to
f43e0f9
Compare
Squashed and ready for merge :) |
If I'm not wrong, this is your first PR to MuseScore ! Thank you and welcome to the contributor community ! |
@lasconic - it certainly is! I'm looking forward to making many more, I hope :) |
https://musescore.org/en/node/274521