Bug 5278 - WARNING: Malformed viewbox string: (l: 0)
Summary: WARNING: Malformed viewbox string: (l: 0)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P4 Normal
Assignee: Eric Seidel (no email)
URL: http://www.carto.net/papers/svg/sampl...
Keywords:
Depends on:
Blocks:
 
Reported: 2005-10-05 16:01 PDT by Julien Palmas
Modified: 2005-10-11 04:01 PDT (History)
0 users

See Also:


Attachments
Reduced test case. (443 bytes, image/svg+xml)
2005-10-05 18:04 PDT, Eric Seidel (no email)
no flags Details
Another reduced test case (293 bytes, text/plain)
2005-10-06 07:02 PDT, Julien Palmas
no flags Details
rwlbuis patch (1.66 KB, patch)
2005-10-10 13:28 PDT, Julien Palmas
eric: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Palmas 2005-10-05 16:01:23 PDT
The URL points to a svg that triggers this warning.
This bug might already exist in the bugzilla list, but could not find where...
Comment 1 Eric Seidel (no email) 2005-10-05 18:04:30 PDT
Created attachment 4227 [details]
Reduced test case.

the use of <symbol> seems to be related.  This test might not be fully reduced
yet though.
Comment 2 Julien Palmas 2005-10-06 07:02:49 PDT
Created attachment 4234 [details]
Another reduced test case

This test case also triggers the warning.

If you add a viewBox attribute to the symbol element, the warning disappears .
For example :
<symbol id="coniferous" viewBox="0 0 100 50">

The warning comes from
KSVG::SVGFitToViewBoxImpl::parseViewBox(KDOM::DOMStringImpl *str)

The viewBox attribute is not compulsory (I think), but the parseViewBox method
is always called. This should be modified.
Comment 3 Julien Palmas 2005-10-06 07:06:19 PDT
Ah, forgot something.

If you add the viewBox attribute like I said before, the warning disappears, but the <use> element is not 
rendered anymore ...
Comment 4 Julien Palmas 2005-10-10 13:28:26 PDT
Created attachment 4299 [details]
rwlbuis patch
Comment 5 Eric Seidel (no email) 2005-10-10 14:03:44 PDT
Comment on attachment 4299 [details]
rwlbuis patch

I'm not sure I really like this "dummy" (hack) SVG element for use, but at
least this patch fixes the hack.
Comment 6 Rob Buis 2005-10-11 04:01:38 PDT
It seems there is still a problem with the reduced testcase.
My suspicion, aspect ratio is not correctly handled!
I'll look at it tonight, unless someone beats me to it :)