Bug 12210

Summary: <svg> does not properly handle float: left?
Product: WebKit Reporter: Nicholas Shanks <nickshanks>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal Keywords: HasReduction
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.intertwingly.net/blog/2006/12/01/The-White-Pebble
Attachments:
Description Flags
test case
none
Test case that illustrates float problem as well as margin, border and padding issues.
none
the fix (and test case) hyatt: review+

Nicholas Shanks
Reported 2007-01-11 07:11:50 PST
I saw this linked to from the WHATWG mailing list with the comment: "That page works well with Opera and Firefox, and degrades gracefully in Safari 2 and Internet Explorer. It is not working well in recent WebKit nightlies, but I'm sure that is temporary." And sure enough the bubbles seem to have floated to the top of the page :)
Attachments
test case (365 bytes, application/xhtml+xml)
2007-01-14 22:23 PST, Eric Seidel (no email) 		no flags
Details
Test case that illustrates float problem as well as margin, border and padding issues. (914 bytes, application/xhtml+xml)
2007-01-14 22:34 PST, Dave Hyatt 		no flags
Details
the fix (and test case) (40.29 KB, patch)
2007-01-14 23:06 PST, Eric Seidel (no email) 		hyatt: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2007-01-14 21:38:31 PST
I think this is that <svg> doesn't properly handle float:left. I expect it's being passed in values for tx, ty when drawing and then ignoring them. Not sure.
Eric Seidel (no email)
Comment 2 2007-01-14 22:23:24 PST
Created attachment 12440 [details] test case
Dave Hyatt
Comment 3 2007-01-14 22:34:43 PST
Created attachment 12441 [details] Test case that illustrates float problem as well as margin, border and padding issues.
Eric Seidel (no email)
Comment 4 2007-01-14 23:06:12 PST
Created attachment 12442 [details] the fix (and test case)
Dave Hyatt
Comment 5 2007-01-14 23:08:19 PST
Comment on attachment 12442 [details] the fix (and test case) Typo: + // This should only be exist for <svg> renderers there's a stray "be". r=me
Note You need to log in before you can comment on or make changes to this bug.