fix #294082: TAB - Mixing mensural value symbols and beaming in historic tablatures is broken. #5304
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves: https://musescore.org/en/node/294082.
TabDurationSymbol::layout2()
is responsible for setting the_beamLength
for the tab duration symbol, which is used to determine the length of the line to draw. Problem is,TabDurationSymbol::layout2()
is only ever called fromChord::layoutStem()
, andChord::layoutStem()
is not being called at all, except fromChordRest::removeDeleteBeam()
. Therefore,_beamLength
is always equal to 0, which results in zero-length lines being drawn.This adds a call to
Chord::layoutStem()
TabDurationSymbol::layout2()
fromLayoutContext::collectPage()
Chord::layout2()
, so that_beamLength
can be set to the correct value.