Re: Status on 1474

[Keith OHara]

Carl Sorensen <c_sorensen <at> byu.edu> writes:

> So what is the current status on 1474?
> 
> I think that the latest proposal from Keith O'Hara was to revert ee0488.
> 
Issue 1474 looks like intentional behavior to me, and non-regressive to 
Janek.   We'll see what Bernard says after he gives this a fresh look with a 
clear mind.

Reverting ee0488 (fixed 1120) looked fine at first, but with the example at the 
bottom, reverting ee0488 results in lyrics stretching the note spacing, the 
essence of 1120.   

Lyrics 'padding needs to be greater than the sum of effective extra-spacing-
heights of the LyricsText plus the relevant Notehead, or Stem, or etc.,  so 
that the NoteColumns can slide ('tuck') over the Lyrics.

ee0488 made this happen by removing a default extra-spacing-height and giving 
extra-spacing-height explicitly to the items who need it.  This had side-
effects.  KeySignatures used extra-spacing-height (in a perverse way) to keep 
multimeasure rests from colliding (1472).  Accidentals more often got space, to 
which I became accustomed (1474).

We can finish the job of re-distributing extra-spacing-height to complete 
ee0488, or find a better solution for issue 1120 (maybe LyricText extra-spacing 
height '(+1 . -1) ? ).  

My current frustration with redistribution: anything that should slide over 
Lyrics needs zero extra-height, but then those things tuck very closely with 
each other.  

What I most want to determine is whether there is a wiser fix to issue 1120, so 
that spacing rules involving Lyrics do not compete with other spacing desires. 
I am still absorbing the depth of Neil's last comment on the issue
<http://lists.gnu.org/archive/html/bug-lilypond/2011-01/msg00169.html>

==
\score { % essence of issue 1120
  << \context Voice = "v" {
    \stemDown g'~ g' g' 
  }
  \new Lyrics \lyricsto "v" { 
    \lyricmode { Ammmmmm A }
  } >>          
  \layout { \context {
    \Score
      \override PaperColumn #'stencil = #ly:separation-item::print
  }}} #(ly:set-option 'debug-skylines