148411 - [ps] CSS sans-serif prints as serif

Status: RESOLVED FIXED

(Core :: Printing: Output, defect)

Description

[Steve White]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.0+) Gecko/20020531
BuildID:    2002053108

This is Linux Mandrake 8.0 using CUPS system.

If text is styled as "font-family: sans-serif", it will be displayed
correctly in a sans-serif font, but it prints as a serif font.

However, if the font-family is set to monospace, the text is printed
in a monospace font, and if the font-family is set to some explicit
family, such as helvetica, it also prints correctly.

In previous versions (NS 6.2 and before), all CSS-styled text would print in a
serif font.


Reproducible: Always
Steps to Reproduce:
1.go to page with text with attribute style="font-family: sans-serif"
2.print page.
3.

Actual Results:  The text is printed as serif font (although other formatting, 
such as size and weight, are correctly printed)

Expected Results:  Should have printed in sans-serif face, shouldn't it?

Comment 1

[Kai Lahmann (is there, where MNG is)]

Linux build 2002052421 here:

print preview -> sans-serif (as written on the textpage)
printing -> serif (and so there's the bug)

Comment 2

[Kai Lahmann (is there, where MNG is)]

no dup found, so confirm..

Comment 3

[Roland Mainz]

Attachment: Testcase (same as http://www.goldang.org/~white/docs/computers/browsers/fontprinting.htm)

Comment 4

[Roland Mainz]

Attachment: Testcase printed with Xprint module (300 DPI, DIN-A4)

Reporter:
Does this PS file print OK for you ?

Comment 5

[Steve White]

Yes, Roland, that is what it was supposed to look like.
(The difference would have been in the line "sans-serif 12pt")

It would appear that this shows that the problem isn't in the
Xprint module, but in the PostScript module (I think CUPS uses
Ghostscript to convert PostScript into a printer file.)

Comment 6

[Roland Mainz]

Added item to the release notes
(http://bugzilla.mozilla.org/show_bug.cgi?id=133795#c77) ...

Comment 7

[rods (gone)]

looks like PS issue over to don

Comment 8

[Pete Zha]

Attachment: patch

fix the font name "sans serif" to "sans-serif"
need r=&sr=

Comment 9

[Pete Zha]

taking

Comment 10

[dcone (gone)]

Comment on attachment 91622 [details] [diff] [review]
patch

r=dcone

Comment 11

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 91622 [details] [diff] [review]
patch

sr=bzbarsky

Comment 12

[Asa Dotzler [:asa]]

Comment on attachment 91622 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to 1.1

Comment 13

[Pete Zha]

checked in