Skip to content

Comment on Primer – CSS toolkit and guidelines that power GitHubparent

Comments

This is due to a Chrome bug with how they render Helvetica. We could hack it ourselves, but it's a regression on their end and it'd just be more work for us.

Are you sure it is a bug, and not just a difference in the way the vertical spacing is calculated? Apple changed the way Helvetica's height is calculated in iOS 3.x and for a while had a "legacy font metrics" that were available for their apps, which included MobileSafari until they pulled the trigger in iOS 4.x. I have found every browser and operating system and often different versions of each to calculate these metrics differently, even for a specific given font. I talked to some people at WebKit and on Safari about this, and they absolutely do not consider it a bug.

Yup, see my comment above or below this. I'd find the tweets about it, but blah, Twitter search.

This is great to know - thank you! And keep up the good work.

Did you file a bug?

We got Paul Irish to re-open the bug after they made the change to their font rendering. It is indeed a bug and has been fixed.

AboutSource Built by g1lg1l

Hackerly is an independent reader for Hacker News, built on the public HN API. Not affiliated with Y Combinator.