<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">That totally works for me.<div><br></div><div>-Sam</div><div><br><div><div>On Jun 13, 2011, at 10:52 AM, Darin Fisher wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Good point!  Maybe we can use a term that is derived from the name of the spec?  ENABLE_CSS3_FLEXBOX?<div><br><div>-Darin<br><br><div class="gmail_quote">On Mon, Jun 13, 2011 at 10:50 AM, Simon Fraser <span dir="ltr"><<a href="mailto:simon.fraser@apple.com">simon.fraser@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word">Using terms like 'new' in code is rarely a good idea. In a year, the context has gone, and 'new' no longer means anything.<div>
<br></div><div><font color="#888888">Simon</font><div><div></div><div class="h5"><br><div><br><div><div>On Jun 13, 2011, at 9:38 AM, Tony Chang wrote:</div><br><blockquote type="cite">Err, ENABLE_NEW_FLEXBOX.<br><br><div class="gmail_quote">
On Mon, Jun 13, 2011 at 9:37 AM, Tony Chang <span dir="ltr"><<a href="mailto:tony@chromium.org" target="_blank">tony@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Sure, no problem.  I'll rename it to ENALBE_NEW_FLEXBOX.<div><div></div><div><br><br><div class="gmail_quote">On Fri, Jun 10, 2011 at 6:12 PM, Sam Weinig <span dir="ltr"><<a href="mailto:weinig@apple.com" target="_blank">weinig@apple.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Since it is confusing to me (and may be to others), perhaps a different name than ENABLE_FLEXBOX should be used, given that we already have flexbox. Maybe, ENABLE_NEW_FLEXBOX?<div>


<br></div><font color="#888888"><div>-Sam</div></font><div><div></div><div><div>  <br><div><div>On Jun 10, 2011, at 2:42 PM, David Hyatt wrote:</div><br><blockquote type="cite"><div style="word-wrap:break-word">
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">


<div><div><div><div><div>On Jun 8, 2011, at 10:57 AM, Tony Chang wrote:</div><br><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/" target="_blank">http://www.w3.org/TR/css3-flexbox/</a> (<a href="http://dev.w3.org/csswg/css3-flexbox/#negative-flexibility" target="_blank">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" target="_blank">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" target="_blank">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/" target="_blank">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" target="_blank">webkit-dev@lists.webkit.org</a><br><a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">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" target="_blank">webkit-dev@lists.webkit.org</a><br>


<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br></blockquote></div><br></div></div></div>_______________________________________________<br>


webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br><a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>


</blockquote></div><br></div></div></div></div><br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></blockquote></div><br>
</div></div></blockquote></div><br>
_______________________________________________<br>webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br><a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
</blockquote></div><br></div></div></div></div></div>
<br>_______________________________________________<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" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></blockquote></div><br></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></body></html>