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 http://trac.webkit.org/changeset/166316
Thumbs up for our shiny shape-outside! :-) Cheers, Zoltan On Tue, Mar 25, 2014 at 12:59 PM, Zoltan Horvath <[email protected]> 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 > >
_______________________________________________ webkit-dev mailing list [email protected] https://lists.webkit.org/mailman/listinfo/webkit-dev
