WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 30342
18985
[QT][CAIRO] x- and y-attributes of <use>-element not used
https://bugs.webkit.org/show_bug.cgi?id=18985
Summary
[QT][CAIRO] x- and y-attributes of <use>-element not used
Dirk Schulze
Reported
2008-05-10 07:17:28 PDT
If I understood the code right, the <use>-element is transformed to a <g>-element (in a basic case). All Attributes are given to the transformed element (like transform,fill,stroke), except for the x- and y-attributes. This only affects the Qt- and Cairo-implementation.
Attachments
Add attachment
proposed patch, testcase, etc.
Dirk Schulze
Comment 1
2008-05-10 12:11:31 PDT
a simple problem:
http://trac.webkit.org/browser/trunk/WebCore/svg/SVGUseElement.cpp#L323
I get a comma instead of a point for floating-point (non-english-speaker), thats why webkit ignores the transformation. e.g.: transform(12,000, 30,500) instead of transform(12.000, 30.500)
Dirk Schulze
Comment 2
2008-05-10 12:13:31 PDT
(In reply to
comment #1
) I use gcc (GCC) 4.2.3
Mark Rowe (bdash)
Comment 3
2008-05-10 23:38:05 PDT
Looks like vsnprintf is locale-aware, which is probably not what we want inside WebCore.
Mark Rowe (bdash)
Comment 4
2008-05-13 11:28:08 PDT
***
Bug 19026
has been marked as a duplicate of this bug. ***
Dirk Schulze
Comment 5
2008-05-18 12:43:34 PDT
A solution could be setlocale(LC_NUMERIC, "C"); add the line before a vsnprintf() operation or system wide (don't know it this is possible) and it should work. (perhaps with #include <local.h>) I tested it in SVGUseElement.cpp with success.
Alexey Proskuryakov
Comment 6
2008-05-20 00:03:15 PDT
See also:
bug 18994
.
Simon Hausmann
Comment 7
2008-06-16 05:42:06 PDT
I think the code should not use String::format() but simply toDouble() on a constructed string and construct the entire string to parse manually. String::toDouble() is locale-unaware as far as I can see.
Dirk Schulze
Comment 8
2009-11-18 01:03:28 PST
*** This bug has been marked as a duplicate of
bug 30342
***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug