Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 11 years ago

#19145 closed defect (bug) (fixed)

Helvetica Neue broken in Firefox on Windows 7

Reported by: toscho's profile toscho Owned by: koopersmith's profile koopersmith
Milestone: 3.3 Priority: high
Severity: normal Version: 3.3
Component: UI Keywords: has-patch
Focuses: Cc:

Description

Helvetica Neue is used now for the admin bar and the headings. It doesn’t look clean in Firefox, see screenshot. Since there is no font with the exact name »Helvetica Neue« on my system, Firefox seems to use a font with a similar name like »Helvetica 65 Medium«. Other browsers use just the default sans-serif font, which is fine. I suggest to remove the Helvetica from the font stack and to use just the generic family »sans-serif«.

OS: Windows 7, 64 bit
Installed Fonts: Helvetica Neue from Adobe and Linotype

Attachments (4)

firefox-helvetica-neue.png (56.9 KB) - added by toscho 13 years ago.
Firefox Aurora 9.0a2 (2011-11-03) vs. Opera 11.52
19145.patch (965 bytes) - added by SergeyBiryukov 13 years ago.
19145.2.patch (918 bytes) - added by SergeyBiryukov 13 years ago.
19145.admin-bar.patch (852 bytes) - added by SergeyBiryukov 13 years ago.

Download all attachments as: .zip

Change History (17)

@toscho
13 years ago

Firefox Aurora 9.0a2 (2011-11-03) vs. Opera 11.52

#1 @ocean90
13 years ago

Other browsers use just the default sans-serif font, which is fine.

Isn't it then a browser bug? Why Firefox choose another font?

#2 @toscho
13 years ago

I’m not sure if Firefox’ behavior really violates the Font matching algorithm. It is a variant of Helvetica Neue, it just doesn’t look good.

#3 @linuxologos
13 years ago

Aurora is an early version of Firefox. Are we sure it will have the same behavior when stable?

Firefox 7.0.1 does not seem to have such problems.

#4 @SergeyBiryukov
13 years ago

Looks like this was missed in #18020?

#5 @SergeyBiryukov
13 years ago

  • Keywords has-patch added

19145.patch would allow those classes to inherit the generic sans-serif family from body.

19145.2.patch just removes Helvetica from the font stack, without removing font-family.

#6 follow-up: @ocean90
13 years ago

  • Keywords ui-feedback added

SergeyBiryukov, Admin Bar is missing in your patch.

#7 in reply to: ↑ 6 @SergeyBiryukov
13 years ago

Replying to ocean90:

Admin Bar is missing in your patch.

Thanks, added 19145.admin-bar.patch.

#8 @ocean90
13 years ago

  • Milestone changed from Awaiting Review to 3.3

Background #18020. And with the new welcome screen it comes back. Needs review.

#9 @azaozz
13 years ago

  • Owner set to koopersmith
  • Status changed from new to assigned

Not sure if this was intentional, assigning to @koopersmith for confirmation.

#10 @ryan
13 years ago

  • Priority changed from normal to high

#11 @koopersmith
13 years ago

  • Keywords ui-feedback removed
  • Resolution set to maybelater
  • Status changed from assigned to closed

This was intentional — we use Helvetica Neue for accent headings (most noticeably, the h2s) and sans-serif for the rest of the admin. I think that by adding some font-weight declarations, recent versions of FF will fall in line. From that screenshot, it also looks like the admin bar is also affected; if it is, that should be fixed.

After speaking to MT, I'm inclined to think that this is a browser bug in the FF Aurora build, and that we should stay the (neue) course. If it's not, let's reopen the ticket and reëvaluate.

#12 @koopersmith
13 years ago

In [19221]:

Remove Helvetica Neue from the admin bar font stack. see #19145.

#13 @SergeyBiryukov
11 years ago

  • Resolution changed from maybelater to fixed
Note: See TracTickets for help on using tickets.