Bug 92661 - flexbox should avoid floats
Summary: flexbox should avoid floats
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tony Chang
URL: http://plexode.com/u/#A9float28%3A27%...
Depends on:
Blocks: 62048
  Show dependency treegraph
Reported: 2012-07-30 11:05 PDT by Tony Chang
Modified: 2012-07-31 18:31 PDT (History)
4 users (show)

See Also:

Patch (3.21 KB, patch)
2012-07-31 13:53 PDT, Tony Chang
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from gce-cr-linux-01 (310.40 KB, application/zip)
2012-07-31 14:28 PDT, WebKit Review Bot
no flags Details
Patch for landing (3.56 KB, patch)
2012-07-31 14:32 PDT, Tony Chang
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Chang 2012-07-30 11:05:53 PDT
Currently if a flexbox comes after a floated item, it's overlapping the float rather than being positioned around it.  See the URL for an example.  The flexbox should avoid the float. For example, change the display to -webkit-box (old flexbox) or table and you can see how the box avoids the float.
Comment 1 Tony Chang 2012-07-31 13:53:49 PDT
Created attachment 155627 [details]
Comment 2 WebKit Review Bot 2012-07-31 14:28:31 PDT
Comment on attachment 155627 [details]

Attachment 155627 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/13393784

New failing tests:
Comment 3 WebKit Review Bot 2012-07-31 14:28:34 PDT
Created attachment 155635 [details]
Archive of layout-test-results from gce-cr-linux-01

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: gce-cr-linux-01  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.39-gcg-201203291735-x86_64-with-Ubuntu-10.04-lucid
Comment 4 Tony Chang 2012-07-31 14:32:38 PDT
Created attachment 155636 [details]
Patch for landing
Comment 5 Tony Chang 2012-07-31 14:51:46 PDT
For completeness, I tried to see if we needed to update RenderBox::avoidsFloats to include new flexbox items, but I couldn't get that test to crash in either new or old flexbox.  We probably fixed something else in old flexbox that causes the test to work.
Comment 6 WebKit Review Bot 2012-07-31 18:31:44 PDT
Comment on attachment 155636 [details]
Patch for landing

Clearing flags on attachment: 155636

Committed r124279: <http://trac.webkit.org/changeset/124279>
Comment 7 WebKit Review Bot 2012-07-31 18:31:47 PDT
All reviewed patches have been landed.  Closing bug.