lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Issue 2245: always align dynamics and lyrics on "main" notehead (iss


From: janek . lilypond
Subject: Re: Issue 2245: always align dynamics and lyrics on "main" notehead (issue 108270044 by address@hidden)
Date: Tue, 08 Jul 2014 07:22:28 +0000

Pushed as

commit 59a842eba0f7ad78289a58a7dfa8fa786cdf11ed
Author: Janek Warchoł <address@hidden>
Date:   Wed Mar 27 17:20:03 2013 +0100

    Issue 2245: always align dynamics and lyrics on "main" notehead

    Until now, LyricTexts and DynamicTexts had their X-parents set to
    the first NoteHead in the NoteColumn.  This resulted in inconsistent
    alignment - placement of lyrics and dynamics depended on the order
    of notes in the input:

    % this was aligned differently
    { <f' g'>1\p <g' f'>\p }
    \addlyrics { la la }

    By using NoteColumns themselves as the X-parents, we make sure that
    the input order won't matter.  Since the NoteColumn contains all
NoteHeads
    (including suspended ones, which usually should be ignored when
aligning),
    as well as Flags and some other objects, we cannot use its X-extent
directly -
    instead, we add a function for calculating X-extent of the "main"
part of the
    NoteColumn, i.e. X-extent of the non-suspended NoteHeads
(represented by the
    NoteHead furthest away from the stem).


Thanks for review!


https://codereview.appspot.com/108270044/

reply via email to

[Prev in Thread] Current Thread [Next in Thread]