Bug 23198 - border-image should not be clipped by border-radius
Summary: border-image should not be clipped by border-radius
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 525.x (Safari 3.2)
Hardware: PC Windows Vista
: P3 Normal
Assignee: Dave Hyatt
URL:
Keywords: HasReduction, InRadar
Depends on:
Blocks:
 
Reported: 2009-01-08 15:03 PST by Lea Verou
Modified: 2009-03-25 10:07 PDT (History)
2 users (show)

See Also:


Attachments
Testcase for bug #23198 (610 bytes, text/html)
2009-01-08 15:04 PST, Lea Verou
no flags Details
Patch to stop clipping border-image (52.45 KB, patch)
2009-03-25 09:53 PDT, Dave Hyatt
eric: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lea Verou 2009-01-08 15:03:36 PST
Background is not clipped according to border-radius, when border-image is also set. This happens even with a solid color background.

For testcase, look at the attached html file (will be uploaded shortly after submission).
Comment 1 Lea Verou 2009-01-08 15:04:39 PST
Created attachment 26540 [details]
Testcase for bug #23198
Comment 2 Mark Rowe (bdash) 2009-01-08 16:12:33 PST
<rdar://problem/6482825>
Comment 3 Dave Hyatt 2009-03-25 09:08:50 PDT
From the latest draft of the spec:

"Backgrounds, but not the border-image, are clipped to the inner, resp., outer curve of the border if ‘background-clip’ is ‘padding-box’ resp., ‘border-box’. Other effects that clip to the border or padding edge (such as ‘overflow’) also must clip to the curve."

So what's making this look like a bug to you is that the border-image got incorrectly clipped to the curve.  Therefore its 4px corners aren't visible, and it looks like the background is running up too far to the edge.

Comment 4 Dave Hyatt 2009-03-25 09:53:28 PDT
Created attachment 28929 [details]
Patch to stop clipping border-image

The test case in this bug actually uses a border-image that is purely an alpha component, so even with this change to match the spec, the test case will still look exactly the same (since the underlying background clips to the outer border radius edge).
Comment 5 Eric Seidel (no email) 2009-03-25 10:00:18 PDT
Comment on attachment 28929 [details]
Patch to stop clipping border-image

So lame.  Can't we lobby www-style?  The code looks fine, but I prefer the old behavior.
Comment 6 Dave Hyatt 2009-03-25 10:07:57 PDT
Fixed in r41976.  I'll continue to complain on www-style.