After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 665648 - SVG contains invalid CSS, makes text clipped in Firefox and Opera
SVG contains invalid CSS, makes text clipped in Firefox and Opera
Status: RESOLVED FIXED
Product: dia
Classification: Other
Component: exports
0.97.x
Other All
: Normal normal
: 0.97.2
Assigned To: Dia maintainers
Dia maintainers
: 655718 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-12-06 03:05 UTC by oversky0
Modified: 2011-12-17 14:18 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
SVG renderer with font-size as attribute (25.08 KB, image/svg+xml)
2011-12-17 13:58 UTC, Hans Breuer 		Details

Description oversky0 2011-12-06 03:05:58 UTC 
How to repeat:
For this dia file, http://dl.dropbox.com/u/2484898/Test/test.dia
Export it to SVG, http://dl.dropbox.com/u/2484898/Test/test.svg
In Firefox 8.0 and Opera 11.52, the right most text, "Screen" is clipped.
The ending letter "n" is displayed as "r".

Checking with W3C validator

The SVG markup is valid.
http://is.gd/QKNGHR

The CSS embedded in the SVG is not.
http://is.gd/weu1Et

Discussion with Firefox 
https://bugzilla.mozilla.org/show_bug.cgi?id=707071
Comment 1 Hans Breuer 2011-12-17 13:58:51 UTC 
Created attachment 203727 [details]
SVG renderer with font-size as attribute

Thanks for bringing this up again (see bug 655718 for initial report).
But this time the discussion at Firefox brought up the IMO right solution.
Rather than adding some random absolute unit within the style attribute,
I've changed the font-size to be in it's own attribute. This way the value
can be calculated like all the other sizes in the file and it still
works with Dia and Inkscape. But finally also with Firefox.
Comment 2 Hans Breuer 2011-12-17 14:14:03 UTC 
Fix pushed to dia-0-97 branch, see:
http://git.gnome.org/browse/dia/commit/?h=dia-0-97&id=80d8d5ec9e9668ea3fbb979fdcf3bf1d7cee57ac
Comment 3 Hans Breuer 2011-12-17 14:18:04 UTC 
*** Bug 655718 has been marked as a duplicate of this bug. ***