[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Caches the interior skylines of vertical axis groups and systems.(is
From: |
Trevor Daniels |
Subject: |
Re: Caches the interior skylines of vertical axis groups and systems.(issue 7185044) |
Date: |
Tue, 19 Feb 2013 16:25:59 -0000 |
From: <address@hidden>
Sent: Tuesday, February 19, 2013 2:22 PM
>
>On 19 févr. 2013, at 16:11, Trevor Daniels <address@hidden> wrote:
>>
>> From: <address@hidden>
>> Sent: Monday, February 18, 2013 8:32 PM
>>
>>> On 18 févr. 2013, at 20:07, address@hidden wrote:
>>>
>>>> You can't just throw functionality overboard when you are "improving"
>>>> things and tell people they have to revert to the old code if they
>>>> care about that functionality. After all, it is totally unclear how
>>>> elements with the old callbacks and elements with the new
>>>> outside-staff-priority ignoring callbacks will even combine.
>>>
>>> It is true that this breaks old functionality for user overrides.
>>>
>>> The goal is certainly not to mask things. I will make sure to put this in
>>> the change log and will write a not-smart convert-ly rule in my next patch
>>> set.
>>
>> That's not good enough. The present overrides of outside-staff-priority
>> figure quite extensively in the user documentation. This will all need to be
>> re-written as part of an acceptable patch to show users how to rearrange
>> the order in which outside-staff objects are placed.
>>
> Thanks for giving me input from a documentation point of view. When
> writing the patch, this was one of the fundamental things I took into
> account. All of the examples in the notation reference, learning manual and
> snippets show no change with this patch applied.
OK, Mike, this is reassuring. Seems like I read too much into
"It is true that this breaks old functionality for user overrides."
> The main thing this patch effects are esoteric grobs that normally don't get
> outside-staff-priority, like MultiMeasureRest, NoteHeads and Rests.
One of these that is commonly given an outside-staff-priority is Slur. This
is used as an example in the LM.
> In light of this, what type of additions to the NR do you feel would be
> necessary
> to indicate the new behavior?
Probably nothing the the NR (although I haven't read it carefully to check)
but have a look at this section in the LM, particularly the bit on Slurs, to
see if
the text, as well as the examples, would remain correct.
http://www.lilypond.org/doc/v2.17/Documentation/learning/outside_002dstaff-objects#the-outside_002dstaff_002dpriority-property
Trevor
- Re: Caches the interior skylines of vertical axis groups and systems. (issue 7185044), (continued)
Re: Caches the interior skylines of vertical axis groups and systems. (issue 7185044), dak, 2013/02/19
Re: Caches the interior skylines of vertical axis groups and systems. (issue 7185044), dak, 2013/02/23
Re: Caches the interior skylines of vertical axis groups and systems. (issue 7185044), dak, 2013/02/23
Re: Caches the interior skylines of vertical axis groups and systems. (issue 7185044), tdanielsmusic, 2013/02/23
Re: Caches the interior skylines of vertical axis groups and systems. (issue 7185044), k-ohara5a5a, 2013/02/24
Re: Caches the interior skylines of vertical axis groups and systems. (issue 7185044), dak, 2013/02/24
Re: Caches the interior skylines of vertical axis groups and systems. (issue 7185044), dak, 2013/02/24