[Webkit-unassigned] [Bug 102538] Introduce a new renderer for grid areas

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=102538





--- Comment #2 from Julien Chaffraix <jchaffraix at webkit.org>  2012-11-16 15:07:09 PST ---
(In reply to comment #1)
> I've thought about this more since we last talked about it and I think we probably don't want anonymous grid areas. Before, I was confused and thought containing block implied it would also be a positioning root. But, since you can't style the grid area, it can never be a positioning root.

You are right, they don't impact positioned elements.

> I think we'll end up with something easier to manage that uses less memory and performs better if we avoid creating grid area renderers since they are only there for positioning/sizing the grid items.
> 
> The only complexity here is that we need to make sure to resolve percentage/calc values using the grid-area as the containing block. We will already need to do this calculation for the margins, width and height. So, I think the only complexity added by not having grid area renderers is that we'll also need to do so for percentage paddings.

I will have to think / prototype your idea to see if I agree. My main concern with that is that your proposal could end up peppering grid knowledge all over the render tree or force us to do horrible hacks in RenderGrid (the grid items' containing block per your proposal) to properly resolve our sizes.

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