Bug 121843 - [CSS Regions] Propagating margins fails when there's a <p> inside the flowed <div>
Summary: [CSS Regions] Propagating margins fails when there's a <p> inside the flowed ...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 57312
  Show dependency treegraph
 
Reported: 2013-09-24 09:22 PDT by Mihai Maerean
Modified: 2013-10-01 04:51 PDT (History)
1 user (show)

See Also:


Attachments
patch to delete the expected png(s) and to add the -expected.html file (75.95 KB, patch)
2013-09-24 09:22 PDT, Mihai Maerean
no flags Details | Formatted Diff | Diff
test with regions (1.27 KB, text/html)
2013-09-25 02:02 PDT, Mihai Maerean
no flags Details
html without regions (1.04 KB, text/html)
2013-09-25 02:03 PDT, Mihai Maerean
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mihai Maerean 2013-09-24 09:22:41 PDT
Created attachment 212474 [details]
patch to delete the expected png(s) and to add the -expected.html file

When there's a <p> inside <div style="flow-into:flow">, the margins of the paragraph should collapse with the parent div but they don't.

The LayoutTests/fast/regions/flow-content-basic.html test can't be turned into a ref test.
The current expected png results are not correct.

This bug blocks bug #120457.
Comment 1 Mihai Maerean 2013-09-25 02:02:37 PDT
Created attachment 212542 [details]
test with regions
Comment 2 Mihai Maerean 2013-09-25 02:03:50 PDT
Created attachment 212543 [details]
html without regions
Comment 3 Mihai Maerean 2013-10-01 04:51:25 PDT
This bug is invalid because:

1. In CSS, the adjoining margins of two or more boxes (which might or might not be siblings) can combine to form a single margin. Two margins are adjoining if and only if both belong to in-flow block-level boxes that participate in the same block formatting context ( http://www.w3.org/TR/CSS2/box.html#collapsing-margins )

2. CSS Regions establish a new block formatting Context. ( http://dev.w3.org/csswg/css-regions/ )