Bug 49913

Summary:

CSS box shadow with percent radius should compute radius based on shadow's own width

Product:

WebKit

Reporter:

Andreas Kling <kling>

Component:

Layout and Rendering

Assignee:

Renata Hodovan <rhodovan.u-szeged>

Status:

RESOLVED DUPLICATE

Severity:

Normal

CC:

bdakin, morrita

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Simple page illustrating the problem	none
WebKit's rendering of test page	none
Firefox4's rendering of test page	none
repro with px.	none

Andreas Kling

Reported 2010-11-22 09:06:55 PST

A cursory glance points at RenderBoxModelObject::paintBoxShadow() which uses the values from RenderStyle::getBorderRadiiForRect() for the shadow as well. If the border-radius is specified in percent, those values should be relative to the shadow's size (with spread taken into account), not the element casting the shadow. Screenshots from WebKit and Firefox4 coming..

Attachments
Simple page illustrating the problem (661 bytes, text/html) 2010-11-22 09:07 PST, Andreas Kling	no flags	Details
WebKit's rendering of test page (12.75 KB, image/png) 2010-11-22 09:08 PST, Andreas Kling	no flags	Details
Firefox4's rendering of test page (20.18 KB, image/png) 2010-11-22 09:08 PST, Andreas Kling	no flags	Details
repro with px. (1006 bytes, text/html) 2010-12-23 21:49 PST, Hajime Morrita	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Andreas Kling

Comment 1 2010-11-22 09:07:47 PST

Created attachment 74559 [details] Simple page illustrating the problem

Andreas Kling

Comment 2 2010-11-22 09:08:20 PST

Created attachment 74560 [details] WebKit's rendering of test page

Andreas Kling

Comment 3 2010-11-22 09:08:59 PST

Created attachment 74561 [details] Firefox4's rendering of test page

Hajime Morrita

Comment 4 2010-12-23 21:49:08 PST

Created attachment 77400 [details] repro with px. This isn't limited for percentage value. other values are should be also scaled accordingly because box-radius is for "box", not shadow. Mozilla and Opera do scale.

Hajime Morrita

Comment 5 2010-12-23 22:15:45 PST

Ah, I realized that this is a dup of Bug 49726 *** This bug has been marked as a duplicate of bug 49726 ***

Hajime Morrita

Comment 6 2011-01-19 23:23:32 PST

Bug 27417 and LayoutTests/fast/box-shadow/spread.html says this is intentional. But I don't think so. Any opinions?

Note You need to log in before you can comment on or make changes to this bug.