WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 weeks ago

#19145 closed defect (bug) (fixed)

Helvetica Neue broken in Firefox on Windows 7

Reported by: toscho Owned by: 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 2 years ago.
Firefox Aurora 9.0a2 (2011-11-03) vs. Opera 11.52
19145.patch (965 bytes) - added by SergeyBiryukov 2 years ago.
19145.2.patch (918 bytes) - added by SergeyBiryukov 2 years ago.
19145.admin-bar.patch (852 bytes) - added by SergeyBiryukov 2 years ago.

Download all attachments as: .zip

Change History (17)

toscho2 years ago

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

comment:1 ocean902 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?

comment:2 toscho2 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.

comment:3 linuxologos2 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.

comment:4 SergeyBiryukov2 years ago

Looks like this was missed in #18020?

SergeyBiryukov2 years ago

SergeyBiryukov2 years ago

comment:5 SergeyBiryukov2 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.

comment:6 follow-up: ocean902 years ago

  • Keywords ui-feedback added

SergeyBiryukov, Admin Bar is missing in your patch.

comment:7 in reply to: ↑ 6 SergeyBiryukov2 years ago

Replying to ocean90:

Admin Bar is missing in your patch.

Thanks, added 19145.admin-bar.patch.

comment:8 ocean902 years ago

  • Milestone changed from Awaiting Review to 3.3

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

comment:9 azaozz2 years ago

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

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

comment:10 ryan2 years ago

  • Priority changed from normal to high

comment:11 koopersmith2 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.

comment:12 koopersmith2 years ago

In [19221]:

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

comment:13 SergeyBiryukov2 weeks ago

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