Re: [Lilypond-auto] Issue 3268 in lilypond: Reinstate text spacing behaviour of 2.16.x

[lilypond]

Updates:
        Status: Started
        Owner: address@hidden
        Cc: address@hidden
        Labels: Patch-new

Comment #1 on issue 3268 by address@hidden: Reinstate text spacing  
behaviour of 2.16.x
http://code.google.com/p/lilypond/issues/detail?id=3268

http://codereview.appspot.com/8009043
Use boxes rather than skyline for text.

This changes several regression tests, each of them slightly, by preventing  
other glyphs from interleaving between and descenders in text and markup.   
See also issue 3207 and issue 2795.

If we can identify any cases where skylines for text is helpful, and should  
be enabled by default (maybe fretboards?) we can enable skylines in those  
cases.

The input for ‘text-script-vertical-skylines.ly’ is changed to shows how  
markup can be fit closely, as an option, if desired.

Anybody who is interested can try out the patch without recompiling  
(allowing testing on non-linux systems) by applying the changes to the .scm  
file, or by making the corresponding overrides:
\layout {\context {
    \Score
    \override DynamicText #'vertical-skylines = #'()
    \override MetronomeMark #'vertical-skylines = #'()
    \override MultiMeasureRestText #'vertical-skylines = #'()
    \override LyricText #'vertical-skylines = #'()
    \override LyricHyphen #'vertical-skylines = #'()
    \override TextScript #'vertical-skylines = #'()
}}

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings