miscreation of inline flow boxes in render lists

[David Hyatt]

This is more complicated than posting a single patch, since it took 
multiple iterations and shuffling to repair the bugs.  Probably would 
be easiest to just get you a current post-1.2 WebCore.

dave

On Apr 14, 2004, at 3:00 PM, Leo Savernik wrote:

> Am Mittwoch 14 April 2004 23:05 schrieb David Hyatt:
>> Which code in layout( ) is putting in nil inline boxes? Is this the 
>> box
>> for the marker object?
>
> I cannot tell, but I think so. It only happens for RenderListItem 
> objects. The
> nil inline box has a width of zero, wherever this comes from.
>
>> Note that old Safari code (even in 1.2) would
>> trash the line box tree and not leave it in a sound state (line boxes
>> for the replaced elements like images and the aforementioned marker
>> would get deleted).  In our current code we have fixed this so that 
>> the
>> line box tree is always in a sane state.  Is that what you're seeing?
>
> Can you please post the relevant patch?
>>
> [...]
>>> Well. I have just verified that RenderList::layout does insert the 
>>> nil
>>> inline
>>> boxes, but it does not make the root inline boxes point to the wrong
>>> inline
>>> text boxes.
>
> I furthermore noticed that after the RenderBlock::layout call in
> RenderListItem::layout, the RootInlineBoxes contain nil elements, but 
> are
> otherwise fine.
>
> Interestingly, in the same layout operation, after the 
> RenderBlock::layout
> call in RenderCanvas::layout, the RootInlineBoxes are garbled as I 
> described
> in my first mail.
>
> mfg
>  	Leo
> _______________________________________________
> Khtml-devel at kde.org
> https://mail.kde.org/mailman/listinfo/khtml-devel