Staccato overlaps highly slanted beam on upper voice
Reported version
3.0
Priority
P0 - Critical
Type
Functional
Frequency
Once
Severity
S4 - Minor
Reproducibility
Always
Status
closed
Regression
No
Workaround
Yes
Project
Automatic placement doesn't seem to put staccato markings high up enough in some cases:
See attached score.
Reproduce:
1) add some high, beamed notes in voice 1
2) add some lower notes in voice two so that the beam is forced to go above.
3) add staccatos to the voice 1 notes
Result:
staccatos collide with the beam.
Expected result:
they don't collide with the beam.
Attachment | Size |
---|---|
Sweden.mscz | 32.39 KB |
Fix version
3.0.3
Comments
It's an issue for staccato and tenuto, which are the articulations we try to place close to the note (inside slurs) as per Articulation::layoutCloseToNote().
Articulations are computed in three passes right now, the first of these happens before beaming so stem lengths are not really known. I think probably we are initially placing the layoutCloseToNote() articulations during that first pass (Chord::layoutArticulations()) but not updating them during the second (Chord::layoutArticulations2())
I am not actually sure these need to be three passes anymore, we could possibly get away with calling both layoutArticulations() and layoutArticulations2() at the same time, or just combining them. Seems there must have been some reason I kept that first pass where it was, but since it causes this problem, that needs to be revisited anyhow.
I tried it and it seems to works without causing any obvious problems. Will need to test further, though.
https://github.com/musescore/MuseScore/pull/4711
Fixed in branch master, commit 943e08dfa1
fix #284380: articulations on beamed notes
Fixed in branch master, commit 24aef32654
_Merge pull request #4711 from MarcSabatella/284380-articulation-beam
fix #284380: articulations on beamed notes_
Fixed in branch 3.0.x, commit 8d9bd8d3aa
_Merge pull request #4711 from MarcSabatella/284380-articulation-beam
fix #284380: articulations on beamed notes_
Automatically closed -- issue fixed for 2 weeks with no activity.