47059 - Font for modern skin is FUGLY!

Status: RESOLVED FIXED

(SeaMonkey :: Themes, defect, P4)

Description

[Mike Pinkerton (not reading bugmail)]

The font that gramps picked for the modern font is hideous on MacOS. 
Unfortunately, he checked it in and then went on sabbatical. Someone has to fix 
this, it's embarrasingly ugly.

Comment 1

[Mike Pinkerton (not reading bugmail)]

nominating for beta2, it's an embarrasment. i'm pissed that don made this change 
and then vanished, and it might be too late to do anything about it.

Comment 2

[Jay Patel [:jay]]

Putting on [nsbeta2-] radar. Not critical to beta2. 

Comment 3

[Mike Pinkerton (not reading bugmail)]

ok, well it being beta2- really pisses me off, but what can I do about it? 
nominating for beta3.

did anyone actually LOOK at the ugly font before making this decision?

Comment 4

[johng]

nav triage team:
Reassigning to german to decide if we need to amke this change and if so, choose 
the font we should use.

Comment 5

[Simon Fraser [no longer active]]

There are real problems with the large font that is chosen by default now -- text 
fields in dialog don't show enough content (e.g. URLs in editor dialogs). So 
please choose a smaller font. Perhaps the Mac-standard Geneva 9 or 10 on mac?

Comment 6

[Matthew T (active 1999-2002)]

Geneva 10 is the Mac OS default, not 9.

Comment 7

[Matthew T (active 1999-2002)]

Ok, let me clarify that. Geneva 10 is *usually* the default font. Geneva 9 is 
used in oddball cases like splash screens, registration info, the Launcher, and 
the Moz 4.x interface.

Comment 8

[german]

Geneva was the font chosen. I am not sure why it came out so bad. we looked at 3 

macs to verify that it looked OK. The original motivation was that on some 

platforms like Linux and Mac the default font turned out too small in addition of 

having the wrong font-family (from Windows), which is a long-standing Bug (5236), 

als nsbeta3+'d and assignd to McAfee.

What was done was changing the CSS font-family sequence from nothing to Geneva, 

Tahoma, Helvetica, Arial, sans-serif; meaning Geneva will trigger first on a 

standard Mac. (see checkin at http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=

context&whitespace_mode=show&subdir=mozilla/themes/modern/global&command=

DIFF_FRAMESET&file=global.css&rev1=1.11&rev2=1.12&root=/cvsroot) We did not check 

in any size information, because we could not determine one on short order to fit 

all 3 platforms, but thats why that bug was left often and nsbeta3'd. I'll look 

into it...

Comment 9

[Matthew T (active 1999-2002)]

> We did not check in any size information

That'll be it, then. You need to specify 10 px (note px, not pt).

Comment 10

[german]

We will attempt to take care of this issue while doing the modern cleanup. CC'ing 
Marlon, one of the designers involved in the new modern theme.

Comment 11

[hangas]

Changing to Themes.  Sending to Joe, this will require you to get review from 
German so that nothing is missed.

Comment 12

[Joe Hewitt (gone)]

This bug was filed back in the days before Modern2.  Are these same concerns 
still valid for the new modern skin?

Comment 13

[Mike Pinkerton (not reading bugmail)]

i'd still prefer a geneva9, but.....

Comment 14

[marlon bishop]

We can't specify 9pt Geneva, see bug# 5236.  Can't do font specific sizing.  So 

everybody would have to have 9, and that's not acceptable on other platforms.  If 

only we could implement CSS font-face-adjust, it would work 

Comment 15

[hangas]

Marking nsbeta3- because this was not a P1 or P2 bug.

Comment 16

[Joe Hewitt (gone)]

This is very subjective.  It's a difficult argument as to what the best 
solution is, but we know for sure that we won't be changing fonts in Modern any 
time soon due to the crunch for RTM and the far-reaching implications of this 
change.  Marking future.  I am certain we will resolve this issue after 6.0.

Comment 17

[Joe Hewitt (gone)]

Fixeed - see fix for 16729.