Re: Directional NoteHead Stencil Support

[Marc Hohl]

Am 21.03.2015 um 00:40 schrieb Thomas Morley:
> 2015-03-21 0:09 GMT+01:00 Thomas Morley <address@hidden>:
>
> > The 'arpeggio-property was deleted somewhere during 2.17. and I never
> > managed to get back the info whether a NoteColumn has an arpeggio,
>
> Btw, I tried again to find the commit on the tracker which removed the
> 'arpeggio-property (and probably the reason for it) and again without
> success.

I think it is

5ab8335d106d736335698245af3c1b2b2455aed6


    Adds arpeggio to conditional item grob array.

    The actual function of conditional elements was not correctly reflected
    in the comment above Separation_item::boxes.  This comment led one 
to believe
    that conditional elements were only used when notes with accidentals
    had ties coming to them.  This is not true.  Conditional elements 
are always
    used for right columns when there is something to the left (see
    Spacing_interface::skylines).  They are omitted _only_ when they are
    accidentals with ties coming to them.

    So why do we want an arpeggio to be a conditional element?  There is
    nothing conditional about it (it will always be printed, unlike 
accidentals
    with ties going to them).  It is because conditional elements have the
    double duty of being conditional (i.e. accidentals) AND being factored
    into springs in note spacing (see the long comment in
    Note_spacing::get_spacing) in the calculation of a spring's ideal 
distance.
    Other elements to the left of a note column, like scripts and 
fingerings,
    are only factored into the minimum distance. We want arpeggios to 
factor
    into the ideal distance because otherwise they will be too close to
    left note-columns in tight spacing situations.

    Lastly, there was no reason in the code base to keep a pointer to the
    arpeggio in the note column, so it is removed here.

---
HTH,

Marc



> -Harm
>
> _______________________________________________
> lilypond-user mailing list
> address@hidden
> https://lists.gnu.org/mailman/listinfo/lilypond-user