<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Just so people know, it was my recommendation that they just start with a new renderer and implementation.<div><br></div><div>Some other recommendations I would make here (apologies if they have been implemented already):<div><br></div><div>(1) Rename the current RenderFlexibleBox to put "deprecated" into the name, e.g., RenderDeprecatedFlexibleBox.</div><div><br></div><div>(2) The old flexbox was never patched for vertical writing modes. Please make sure when you build the new renderer from the ground up that you take this into account.</div><div><br></div><div>(3) Please consult with rendering experts for any changes you have to make to base classes, especially RenderBlock and RenderBox. We may be able to help simplify some of that code, especially compared to the old flexbox.</div><div><br></div><div>(4) Use the old flexbox code as a rough guide, but be aware of its issues, e.g., too much layout when flexing, box-ordinal stuff is very slow, etc. I think some of the bugs you tried to fix already should inform this somewhat.</div><div><br></div><div>Definitely keep rendering experts in the loop on this and have fun implementing!</div><div><br></div><div>Dave</div><div><br></div><div><div><div><br></div><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div><div><div><div>On Jun 8, 2011, at 10:57 AM, Tony Chang wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hi webkit-dev,<div><br></div><div><div>I wanted to let you know that Ojan and I plan to add flexbox layout support to WebCore.  WebCore already supports an older flexbox implementation (display: box), but the new spec is designed to be easier for developers to understand and more powerful.  The old flexbox will still remain in WebCore since none of the CSS properties overlap with the new flexbox spec.  The spec can be found at: <a href="http://www.w3.org/TR/css3-flexbox/">http://www.w3.org/TR/css3-flexbox/</a> (<a href="http://dev.w3.org/csswg/css3-flexbox/#negative-flexibility">http://dev.w3.org/csswg/css3-flexbox/</a>)</div>
<div><br></div><div>This support will be behind the ENABLE_FLEXBOX feature define (<a href="https://bugs.webkit.org/show_bug.cgi?id=62049">https://bugs.webkit.org/show_bug.cgi?id=62049</a>) and there is a meta bug tracking the feature's development (<a href="https://bugs.webkit.org/show_bug.cgi?id=62048">https://bugs.webkit.org/show_bug.cgi?id=62048</a>).  I expect this feature to eventually be enabled by all ports.</div>
</div><div><br></div><div>I am ready to setup a buildbot for tracking the compile and flexbox related layout tests.  Should I go ahead and get this added to <a href="http://build.webkit.org/">build.webkit.org</a>'s waterfall?</div>
<div><br></div><div>Thanks,</div><div>Tony</div><div><br></div>
_______________________________________________<br>webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br><a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br></blockquote></div><br></div></div></div></div>_______________________________________________<br>webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev<br></blockquote></div><br></div></div></body></html>