18135 - The default skin ought to use CSS2 systemfonts

Status: VERIFIED FIXED

(SeaMonkey :: General, defect, P3)

Description

[Braden]

The UI fonts under X are tiny unless the logical ppi is cranked up significantly
over the physical ppi. The size is not consistent with the size in the Windows
UI.

Comment 1

[Braden]

Attachment: Screen grab of Mozilla under Linux @ 91 ppi

Comment 2

[Braden]

Attachment: Screen grab of Mozilla under Win98 @ 91 ppi

Comment 3

[Braden]

The screen shots I've attached should illustrate the problem. I realize the
comparison would be easier to make if the font wasn't italicized under Windows,
but I don't know what's making it do that.

Comment 4

[troy]

Looks like a DUP of 5397 to me. It's definitely not a layout bug, that's for
sure. Layout is all XP code and does not decide which physical fonts to use

*** This bug has been marked as a duplicate of 5397 ***

Comment 5

[Chris Petersen]

Agreed. Marking as verified dup of 5397.

Comment 6

[Erik van der Poel]

I don't think this is a dup of 5397. Reopening.

Someone on the Net figured out that our Windows GFX code is using the wrong
font height specification. When I fixed it, the UI fonts ended up looking too
small. So I took a look at global.css, where the base font for all XUL is set.
That font was set to 3mm, which is only 8.5pt (point), and that is indeed very
small.

Comment 7

[Erik van der Poel]

Dave, please take a look at this bug report.

Comment 8

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

The relevant bug on the Windows GFX code is bug 13072, right?

Comment 9

[Erik van der Poel]

Yes, the Windows GFX bug is 13072.

Comment 10

[Braden]

Guys, please also have a look at 18781. I don't think it's a dupe of 13072.

Comment 11

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

*** Bug 21554 has been marked as a duplicate of this bug. ***

Comment 12

[Erik van der Poel]

Moving all of my M15s to M16. Please add comments if you disagree.

Comment 13

[Brent Metzler]

*** Bug 24769 has been marked as a duplicate of this bug. ***

Comment 14

[Pierre Saslawsky]

*** Bug 25469 has been marked as a duplicate of this bug. ***

Comment 15

[Erik van der Poel]

I think the UI skins ought to use CSS system fonts, but they are not implemented
yet. There are several font size issues on Unix (logged as separate bugs), but
when system fonts arrive, I believe this bug will be fixed too, since both the
font name and font size will come from the system, which ought to be better.
The CSS system font bug is bug 3371, so I'm adding the dependency to this bug.
Since the UI skins are owned by the UI group, I'm reassigning this bug to ben,
who may wish to assign to the appropriate person.

Comment 16

[Pierre Saslawsky]

Bug 3371 is fixed. This bug is now blocked by bug 33313 (GTK-GFX code for css 
system fonts must be implemented) and bug 16729 (widgets in chromes should use 
the css system fonts).

Comment 17

[Alan S. Jones]

Is this bug related to 5236
The descriptions are very very close.

Comment 18

[Kathleen :Brade]

*** Bug 36357 has been marked as a duplicate of this bug. ***

Comment 19

[Braden]

When filing this bug, I made the error of noting two distinct issues. The first
of these was that the default UI font in Linux was tiny. In my estimation, that
has been pretty fully addressed for some time now. That problem should only
occur now when the X server is misconfigured, and I think that's what bug 5236
is intended to track at this point.

The second issue was that, at a given size value in CSS, the actual size in
pixels of the rendered font differed notably between Linux and Windows. I'm not
sure of the status on this account, as it's been a long time since I looked at
Mozilla under Windows, but I note that bug 13072 has been fixed.

This bug seems to have morphed into "The default skin ought to use CSS2 system
fonts", which is really an issue orthogonal to both of the points initially
raised.

I think this bug needs a summary change. If font sizing parity between Linux and
Windows has not been achieved, the summary should be trimmed to note only that
issue. If it *has* been achieved, then either this bug should be closed, or its
summary should be changed (along with its component, I imagine) to reflect the
issue of the font in the default skin.

Comment 20

[tenthumbs]

I have to disagree about the statement in the first paragraph about X servers.
Mine is properly configured and I have set Mozilla's screen resolution to match
yet fonts in the UI are far too small to read. They've always been that way.

Comment 21

[Braden]

If your X server is properly configured, you should probably be setting the pref
for logical ppi to '0' so that Mozilla will use the value reported by the X
server. But please move further discussion of this issue to bug 5236.

I am changing the summary and component for this bug. While the topics covered
by the discussion have varied, the change reflects the dependencies that have
been created.

Comment 22

[Ben Goodger (use ben at mozilla dot org for email)]

waiting for native implementations....

Comment 23

[Ben Goodger (use ben at mozilla dot org for email)]

*** This bug has been marked as a duplicate of 16729 ***

Comment 24

[Braden]

This is not a duplicate of 16729. That bug covers all skins shipping with the
browser. This one only covers the default skin.

Comment 25

[Ben Goodger (use ben at mozilla dot org for email)]

the default mozilla skin now uses system fonts. if there are problems with the
system fonts used, please file new bugs on the specific discrepancies.

Comment 26

[Chris Petersen]

Marking verified per last comments.