Bug 18328

Summary: Modifying rx="" or ry="" on a <rect> doesn't relayout
Product: WebKit Reporter: Cameron McCormack (:heycam) <heycam>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
Patch and test for the bug.
none
Patch and test for the bug. mjs: review+

Cameron McCormack (:heycam)
Reported 2008-04-06 00:54:40 PDT
Modifying the rx="" or ry="" attributes on an SVG <rect> element doesn't cause the element to be repainted with the new shape.
Attachments
Patch and test for the bug. (4.06 KB, patch)
2008-04-06 01:02 PDT, Cameron McCormack (:heycam) 		no flags
DetailsFormatted DiffDiff
Patch and test for the bug. (4.04 KB, patch)
2008-04-06 01:08 PDT, Cameron McCormack (:heycam) 		mjs: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Cameron McCormack (:heycam)
Comment 1 2008-04-06 01:02:49 PDT
Created attachment 20360 [details] Patch and test for the bug.
Cameron McCormack (:heycam)
Comment 2 2008-04-06 01:08:54 PDT
Created attachment 20361 [details] Patch and test for the bug. With my e-mail address filled in properly this time.
Eric Seidel (no email)
Comment 3 2008-04-13 22:25:37 PDT
WildFox should peak at this.
Maciej Stachowiak
Comment 4 2008-04-22 22:30:03 PDT
Comment on attachment 20361 [details] Patch and test for the bug. r=me Looks exactly right. (Actually it would be even better if the test covered ry as well as rx but that would be optional.
Nikolas Zimmermann
Comment 5 2008-04-23 03:02:12 PDT
Oh, funny bug. Good you noticed. Exactly because of bugs like that the plan was to fill svg/dynamic-updates/SVGXXXElement-svgdom-xxx-prop.js / etc. with life. Unfortunately we're still at SVGGElement, and it's a long way to SVGRectElement :-)
Mark Rowe (bdash)
Comment 6 2008-04-24 18:01:05 PDT
Landed in r32528.
Note You need to log in before you can comment on or make changes to this bug.