[webkit-dev] Shape-inside removal, bug #130698

Zoltan Horvath zoltan at webkit.org
Wed Mar 26 15:48:34 PDT 2014

Hi there,

The patch is landed in http://trac.webkit.org/changeset/166301
I already did the first step to simplify our geometry codebase in

Thumbs up for our shiny shape-outside! :-)


On Tue, Mar 25, 2014 at 12:59 PM, Zoltan Horvath <zoltan at webkit.org> wrote:

> Hi there,
> CSS Shapes Level 1 (CR [1]) only contains shape-outside. We are focusing our effort on finalizing the implementation of shape-outside, so it's worth to remove shape-inside code at this point for now.
> A list of reasons for the removal:
> - Shape-inside is only part of Shapes Level 2, which needs to be improved on some topics.
> - Shape-inside is lack of new shapes support (e.g. inset).
> - Deprecated shapes (r165472) are removed from the code (e.g. rectangle), which affects shape-inside.
> - The current shape-inside code spreads across the layout code.
> - The current shape-inside implementation is experimental in some areas, and the partially implemented code can have security implications.
> - Removal of shape-inside opens possibilities for code complexity and performance optimizations for shape-outside. (e.g. simpler geometry code)
> I've got the patch reviewed in bug #130698 [2] ready for landing. If there are no objections, I'll land the patch tomorrow morning.
> Cheers,
> Zoltan
> [1] http://www.w3.org/TR/2014/CR-css-shapes-1-20140320/
> [2] https://bugs.webkit.org/show_bug.cgi?id=130698
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-dev/attachments/20140326/59b4a5d9/attachment.html>

More information about the webkit-dev mailing list