[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Adds padding between Hairpins and SpanBars. (issue 5438060)
From: |
address@hidden |
Subject: |
Re: Adds padding between Hairpins and SpanBars. (issue 5438060) |
Date: |
Fri, 27 Jan 2012 20:40:28 +0100 |
On Jan 27, 2012, at 8:10 PM, Keith OHara wrote:
> On 2011/11/28 09:11:47, mike_apollinemike.com wrote:
>> If the hairpins stop before span bars but
>> extend all the way when span bar's don't exist
>> (including when they are not
>> present because of the RemoveEmptyStaffContext),
>> then I'd much rather go with
>> your patch, as it is much less invasive than mine.
>>
>
> Mike,
> Did you need for your own reasons that broken Hairpins stop early if and only
> if any concurrent Hairpin hits a span bar, or did you think somebody else
> wanted that?
>
It was a pure aesthetic judgement on my part - I should have polled people
before but forgot.
> It would be much simpler to simply end all Hairpins the same way, so they
> don't need to check the concurrent Hairpins. I think this is what everyone
> is asking for, but you added a regression test 'hairpin-span-bar.ly' that
> explicitly requires Hairpin lengths to change based on context.
This is probably a one-two liner on/off flag w/ an extra property - I'm
musicking now, so if you have some time & can add it, I'd be much obliged.
Otherwise, I'll be un-musicking in a weekish.
Cheers,
MS