Bug 18328 - Modifying rx="" or ry="" on a <rect> doesn't relayout
Summary: Modifying rx="" or ry="" on a <rect> doesn't relayout
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-06 00:54 PDT by Cameron McCormack (:heycam)
Modified: 2008-04-24 18:01 PDT (History)
1 user (show)

See Also:


Attachments
Patch and test for the bug. (4.06 KB, patch)
2008-04-06 01:02 PDT, Cameron McCormack (:heycam)
no flags Details | Formatted Diff | Diff
Patch and test for the bug. (4.04 KB, patch)
2008-04-06 01:08 PDT, Cameron McCormack (:heycam)
mjs: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Cameron McCormack (:heycam) 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.
Comment 1 Cameron McCormack (:heycam) 2008-04-06 01:02:49 PDT
Created attachment 20360 [details]
Patch and test for the bug.
Comment 2 Cameron McCormack (:heycam) 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.
Comment 3 Eric Seidel (no email) 2008-04-13 22:25:37 PDT
WildFox should peak at this.
Comment 4 Maciej Stachowiak 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.
Comment 5 Nikolas Zimmermann 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 :-)
Comment 6 Mark Rowe (bdash) 2008-04-24 18:01:05 PDT
Landed in r32528.