Summary: | WARNING: Malformed viewbox string: (l: 0) | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Julien Palmas <julien.palmas> | ||||||||
Component: | SVG | Assignee: | Eric Seidel (no email) <eric> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | ||||||||||
Priority: | P4 | ||||||||||
Version: | 420+ | ||||||||||
Hardware: | Mac | ||||||||||
OS: | OS X 10.4 | ||||||||||
URL: | http://www.carto.net/papers/svg/samples/symbol.svg | ||||||||||
Attachments: |
|
Description
Julien Palmas
2005-10-05 16:01:23 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.
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.
Ah, forgot something. If you add the viewBox attribute like I said before, the warning disappears, but the <use> element is not rendered anymore ... Created attachment 4299 [details]
rwlbuis patch
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.
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 :) |