[webkit-reviews] review requested: [Bug 94237] Fix cross-direction stretch for replaced elements in row flexbox : [Attachment 159771] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Aug 21 14:44:49 PDT 2012
Shezan Baig <shezbaig.wk at gmail.com> has asked for review:
Bug 94237: Fix cross-direction stretch for replaced elements in row flexbox
https://bugs.webkit.org/show_bug.cgi?id=94237
Attachment 159771: Patch
https://bugs.webkit.org/attachment.cgi?id=159771&action=review
------- Additional Comments from Shezan Baig <shezbaig.wk at gmail.com>
I think this patch requires another review. I had to disable stretching when
the child and the flexbox had orthogonal flows, in order to make the tests
above pass. The issue here is that the child already gets overrideHeight set
during the flex layout, so we can't use overrideHeight for stretching. I'm not
sure what to do here (do we use overrideWidth instead in this case?). I've
added a FIXME for this, so hopefully it can be addressed in a future patch.
More information about the webkit-reviews
mailing list