[Webkit-unassigned] [Bug 13235] SVG does not render

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=13235


rwlbuis at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1




------- Comment #1 from rwlbuis at gmail.com  2007-04-11 04:16 PDT -------
I confirm that this happens on that url. The calculation of the transform
attribute of the main group element seems messed up. When I look at the used js
code I think it is due to this:

src/gfx/svg.js:

if(dojo.render.html.safari){
        this.rawNode.setAttribute("transform", "matrix(" +
        parseInt(tm.xx) + "," + parseInt(tm.yx) + "," +
        parseInt(tm.xy) + "," + parseInt(tm.yy) + "," +
        parseInt(tm.dx) + "," + parseInt(tm.dy) + ")");
}else{
        this.rawNode.setAttribute("transform", "matrix(" +
        tm.xx.toFixed(8) + "," + tm.yx.toFixed(8) + "," +
        tm.xy.toFixed(8) + "," + tm.yy.toFixed(8) + "," +
        tm.dx.toFixed(8) + "," + tm.dy.toFixed(8) + ")");
}

AFAICS toFixed() works fine in Safari release and ToT, so I don't see the need
for this construction. Moreover parseInt will convert values like 0.9(for
instance a scale factor) to 0. Using the 0 for a/d of the matrix causes the
whole problem. Since FF parseInt and toFixed act the same as Safari I think the
above if branch for Safari should just go.
Cheers,

Rob


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.