Re: Fixes NoteColumn vs SpanBar collisions. (issue 5323062)

[address@hidden]

On Nov 7, 2011, at 9:33 PM, Keith OHara wrote:

> On Mon, 07 Nov 2011 20:51:14 -0800, address@hidden <address@hidden> wrote:
> 
>> I'm OK with getting rid of all of this pure-from-neighbor and span-bar-stub 
>> stuff and just create several SpanBars instead of one SpanBar with gaps in 
>> its stencil.  I can reuse code I've already written, so it's not that bad.
> 
> Would you do it as a revert of 4f49b000 followed by a commit to make a clean 
> fix for issue 910 ?
> 

Exactly.  I'll have time to work on this today.

>> But, my question is: was it ever a good idea to have the pure heights of 
>> span bars calculated the way they were?  SpanBars are inherently cross 
>> staff, so how can they estimate a correct Y-extent without triggering a 
>> VerticalAlignment?
>> 
> 
> The correct height for SpanBars during note-spacing would be whatever 
> corresponds to the tentative staff-spacing used during horizontal spacing -- 
> the spacing table for which you recently implemented caching.  SpanBars did 
> have that as their Y-extent in 2.14.  (But of course they did not have the 
> proper gaps in their skylines, if somebody used 'allow-span-bar=#f)
> 

Ok.