Bug 10524

Summary: svg transform: comma delimiting breaks display
Product: WebKit Reporter: jay <jay>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal Keywords: EasyFix, SVGHitList
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.w3.org/TR/SVG/coords.html#TransformAttribute
Attachments:
Description Flags
A simple test case demonstrating the problem.
none
First attempt darin: review+

jay
Reported 2006-08-23 06:33:12 PDT
<g transform="translate(52,15), scale(.3) "> does not display scale whereas <g transform="translate(52,15) scale(.3) "> does
Attachments
A simple test case demonstrating the problem. (253 bytes, image/svg+xml)
2006-08-23 07:48 PDT, Eric Seidel (no email) 		no flags
Details
First attempt (8.37 KB, patch)
2006-08-23 09:46 PDT, Rob Buis 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2006-08-23 07:45:52 PDT
Pretty fundamental bug. It would be nice to have a test case here though.
Eric Seidel (no email)
Comment 2 2006-08-23 07:48:56 PDT
Created attachment 10178 [details] A simple test case demonstrating the problem.
Eric Seidel (no email)
Comment 3 2006-08-23 07:51:13 PDT
We probably need more test cases to test our parsing: http://www.w3.org/TR/SVG/coords.html#TransformAttribute Marking this as SVG hitlist, since we'd never ship SVG this way (especially since this is so easy to fix).
Rob Buis
Comment 4 2006-08-23 09:46:25 PDT
Created attachment 10184 [details] First attempt This should fix it. Let me know if more transform stuff needs to be tested. Cheers, Rob.
Darin Adler
Comment 5 2006-08-23 22:57:03 PDT
Comment on attachment 10184 [details] First attempt r=me (although mid(1) is an easier way to do what the code is doing with right())
Rob Buis
Comment 6 2006-08-25 02:13:34 PDT
Landed in r16008.
Note You need to log in before you can comment on or make changes to this bug.