Closed
Bug 400150
Opened 18 years ago
Closed 18 years ago
Remove |font-family: sans-serif| from svg.css
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
People
(Reporter: jwatt, Assigned: jwatt)
Details
Attachments
(2 files, 1 obsolete file)
|
1.01 KB,
patch
|
tor
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
|
1.02 KB,
patch
|
tor
:
superreview+
tor
:
approval1.9+
|
Details | Diff | Splinter Review |
We should remove the |font-family: sans-serif| rule from svg.css, or else add a comment saying why we have it. The spec says the initial value depends on the user agent, so we don't have it there for spec reasons. I think it's stupid that the text in mixed HTML/SVG examples shows up in a different font. E.g. https://bugzilla.mozilla.org/attachment.cgi?id=284921
Attachment #285233 -
Flags: review?(tor)
|
Assignee | |
Comment 1•18 years ago
|
||
Attachment #285233 -
Attachment is obsolete: true
Attachment #285234 -
Flags: review?(tor)
Attachment #285233 -
Flags: review?(tor)
|
||
Comment 2•18 years ago
|
||
CSS doesn't support // comments. Use /**/ comments.
Attachment #285234 -
Flags: review?(tor) → review+
|
|
Assignee | |
Comment 3•18 years ago
|
||
Thanks Jesse.
Attachment #285506 -
Flags: superreview?(roc)
Attachment #285506 -
Flags: approval1.9?
Attachment #285234 -
Flags: superreview+
Attachment #285506 -
Flags: superreview?(roc)
Attachment #285506 -
Flags: superreview+
Attachment #285506 -
Flags: approval1.9?
Attachment #285506 -
Flags: approval1.9+
|
|
Assignee | |
Comment 4•18 years ago
|
||
Checked in.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•