[Webkit-unassigned] [Bug 46775] getBoundingClientRect does not work with SVG <text>
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Oct 8 01:50:16 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=46775
--- Comment #9 from Nikolas Zimmermann <zimmermann at kde.org> 2010-10-08 01:50:16 PST ---
(In reply to comment #7)
> Created an attachment (id=70200)
--> (https://bugs.webkit.org/attachment.cgi?id=70200) [details]
> Fix and augmented layout test
>
> The fix is as suggested by Nico, except that I used boundingBoxForElement instead of boundingBoxForSVGElement.
> (Is this not a more appropriate name? I am a little confused here: it's either boundingBoxForElement, to be used in Element; otherwise, should it not have been simply SVGElement::boundingBox? The same naming style is used e.g. in SVGElement::instancesForElement.)
Yeah, just boundingBox is fine.
>
> The <text> fix also includes the zooming fix, previously submitted to 42815.
>
> The layout test is focused on zooming, not on <text>, although it does include a test for <text>, easy to identify on failure (unlike in the previous patch).
>
> There is still no JavaScript-driven zooming. I tried it, and it's flaky: it works most of the time, but not all the time, and the remaining times the coordinates are either off-by-one, or zero, or (rarely) contain garbage values.
> Before adding such tests to zoom/page, I will look into how to make them unflaky.
Okay, I replied to you in private mail, this is really high priority. The "zoom" property is never used for SVG, and what we really needs to work, is <svg viewBox="0 0 100 100"> + zooming/scrolling to work :-)
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list