[Webkit-unassigned] [Bug 12123] REGRESSION: Incomplete repaint of floats' overflows

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=12123


hyatt at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #13050|review?                     |review-
               Flag|                            |




------- Comment #15 from hyatt at apple.com  2007-02-08 21:34 PDT -------
(From update of attachment 13050)
This code in RenderTable:

+    if (paintPhase == PaintPhaseFloat)
+        paintFloats(paintInfo, tx, ty, false);

will (I believe) cause a regression when doing selection dragging, since the
text inside the floats will no longer be painted as part of the selection.

I believe that assigning overflowing floats inside table cells to be painted by
the table will result in potential stacking order regressions if the floats
overlap.

Cells do not paint precisely in document order and yet that's what this float
painting is going to do.

I think now that a different approach is perhaps warranted here.  The issue at
hand seems to be that there are repaint issues with floats because of
horizontal float "overflow".

These sorts of repaint issues have already been solved with normal flow
overflow.  I think a much simpler solution that will avoid forcing tables and
flexible boxes to worry about maintaining floating objects lists would be to
incorporate horizontal spillage from float overflow into m_overflowLeft and
m_overflowWidth.

Vertical float spillage was traditionally not considered overflow simply
because the next block could also contain the float.  However, this is never
the case with horizontal float spillage.  I think you can treat horizontal
float spillage as overflow out of any blocks that the float ends up
intersecting, and you'll end up with a much simpler patch.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.