Invisible line should not be included in skyline

• May 1, 2019 - 20:12
Reported version
3.0
Priority
P2 - Medium
Type
Functional
Frequency
Once
Severity
S4 - Minor
Reproducibility
Always
Status
closed
Regression
No
Workaround
No
Project

In the attached file, you can see in the first measure, the hairpin is avoiding the accent. The second measure show what the default should be based upon the style settings in the file. Here is a picture to help visualize this explanation.

invisible line.PNG

invisible line.mscz


Comments

Priority P2 - Medium

Agreed. We actually just made a change in master that improves how invisible elements are treated overall, but this case is not considered. For text lines with lin not visible, we should not include the invisible line in the skyline calculation, just as you say. I'm working on a PR right now that touches this exact code, so I will see about a fix, but if nothing else, my current code makes it possible to simply move the line up without the need to disable autoplace.

Fix version
3.1.0