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

Fix GH#26494: MusicXML Import - Beams don't have their color imported #26495

Merged
merged 2 commits into from
Feb 14, 2025

Conversation

Jojo-Schmitz
Copy link
Contributor

@Jojo-Schmitz Jojo-Schmitz commented Feb 14, 2025

Resolves: #26494

Current restriction: doesn't work for grace note beams
Further restriction: if in the MusicXML different colors are set for different strokes of a beam, like for 16ths' beams' upper and lower stroke, the last color wins, MuseScore does not allow for this, it colors the entire beam, not single strokes of it.

Also added color import for arpeggios and extended testColor.xml to test both

@Jojo-Schmitz Jojo-Schmitz force-pushed the xml/import-beam-color branch 2 times, most recently from e423c14 to e7b9800 Compare February 14, 2025 12:43
@oktophonie oktophonie requested a review from miiizen February 14, 2025 13:11
@Jojo-Schmitz Jojo-Schmitz force-pushed the xml/import-beam-color branch 3 times, most recently from 2326e00 to 00b3d50 Compare February 14, 2025 13:32
@miiizen miiizen merged commit 1ab5654 into musescore:master Feb 14, 2025
11 checks passed
@Jojo-Schmitz Jojo-Schmitz deleted the xml/import-beam-color branch February 14, 2025 15:25
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 14, 2025
'Backport' of musescore#26495

Current restriction: doesn't work for grace note beams
Further restriction: if in the MusicXML different colors are set for different strokes of a beam, like for 16ths' beams' upper and lower stroke, the last color wins, MuseScore does not allow for this, it colors the entire beam, not single strokes of it.
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 14, 2025
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 14, 2025
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 14, 2025
'Backport' of musescore#26495, 1st commit

Current restriction: doesn't work for grace note beams
Further restriction: if in the MusicXML different colors are set for different strokes of a beam, like for 16ths' beams' upper and lower stroke, the last color wins, MuseScore does not allow for this, it colors the entire beam, not single strokes of it.
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 14, 2025
'Backport' of musescore#26495, 1st commit

Current restriction: doesn't work for grace note beams
Further restriction: if in the MusicXML different colors are set for different strokes of a beam, like for 16ths' beams' upper and lower stroke, the last color wins, MuseScore does not allow for this, it colors the entire beam, not single strokes of it.
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 14, 2025
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 14, 2025
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 14, 2025
Jojo-Schmitz added a commit to Jojo-Schmitz/MuseScore that referenced this pull request Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MusicXML Import - Beams don't have their colour imported
2 participants