Bug 138139

Summary: [CSS Shapes] Content does not wrap with overflow: hidden and reference box different from margin-box
Product: WebKit Reporter: Bear Travis <betravis>
Component: CSSAssignee: Bem Jones-Bey <bjonesbe>
Status: RESOLVED FIXED    
Severity: Normal CC: bjonesbe, commit-queue, esprehn+autocc, glenn, kondapallykalyan, zoltan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 140189    
Attachments:
Description Flags
Test case
none
Simplified Testcase
none
Patch none

Description Bear Travis 2014-10-28 11:19:45 PDT
Created attachment 240558 [details]
Test case

This specific combination of CSS properties causes inline content not to wrap but instead be pushed down below the shaped float.

on the float
border-radius that causes the border-box to be a circle
shape-outside margin-box
margin > 1

on the inline content
overflow hidden

See the attached test case
Comment 1 Bem Jones-Bey 2015-01-06 11:13:58 PST
Created attachment 244077 [details]
Simplified Testcase

It looks like this doesn't even need a border radius. The key here is that the shape needs to be smaller than the margin box of the float and the paragraph to wrap needs to have overflow: hidden. This simplified testcase version doesn't have a margin or a border, just an inset(1em).
Comment 2 Bem Jones-Bey 2015-01-07 10:39:04 PST
Created attachment 244163 [details]
Patch
Comment 3 Dave Hyatt 2015-01-07 11:30:58 PST
Comment on attachment 244163 [details]
Patch

r=me. Vertical writing mode test would be nice though.
Comment 4 Bem Jones-Bey 2015-01-07 11:37:12 PST
(In reply to comment #3)
> Comment on attachment 244163 [details]
> Patch
> 
> r=me. Vertical writing mode test would be nice though.

Thanks. Since this is already in the CQ, I filed Bug 140189 to make sure I add that test.
Comment 5 WebKit Commit Bot 2015-01-07 12:13:08 PST
Comment on attachment 244163 [details]
Patch

Clearing flags on attachment: 244163

Committed r178045: <http://trac.webkit.org/changeset/178045>
Comment 6 WebKit Commit Bot 2015-01-07 12:13:13 PST
All reviewed patches have been landed.  Closing bug.