[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5174 Entire \tempo marking should avoid excessive printing commands

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

• labels: Fixed_2_21.0 --> Fixed_2_21_0

---

[issues:#5174] Entire \tempo marking should avoid excessive printing commands

Status: Fixed
Labels: Fixed_2_21_0
Created: Tue Aug 15, 2017 11:10 PM UTC by Étienne Beaulé
Last Updated: Thu Sep 07, 2017 03:14 PM UTC
Owner: Étienne Beaulé
Attachments:

• KCC.cropped.svg (23.9 kB; image/svg+xml)
• Screenshot.png (54.9 kB; image/png)

Currently, separate components of the tempo tag are held in different <text tags, each with a <tspan without attributes. This is wasteful, and can make text ugly. KCC.cropped.svg shows how this fragmentation causes a bad result (ignoring issue [#3778] with \override Score.MetronomeMark.vertical-skylines = #'()). The text is separated into 5 text blocks, whereas only 2 are necessary (before/after note).

There are a few issues at play here:

• Attributes such as the font-family and style should be in the <tspan tags, and so the opening parenthesis may be combined in the same text tag. This would prevent the excessive spacing seen in the example.
• The equal sign is inexplicably unlinked to the metronome marking. I couldn't find where this goes wrong when reading scm/translation-functions.scm for issue [#4678] as all the components seem to be combined there, and when there is not a range of beat markings (e.g. "4 = 100", not "4 = 100 - 120"), the equal sign is combined with the rest.
• The closing parenthesis is also separated from the rest of the metronome mark, leaving a strange gap as seen, and should be combined with the rest of the text.

I can't seem to find where the issue is occuring for points 2 & 3, but by specifying the position of each component (and problems in concatenating them, the spacing is off and the syntax messier.

Making these changes manually in the example, there were 10 units of space between the indication and metronome mark, and with an odd cause in Lilypond's text handling.

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto