737438 - mozillians-dev fonts are not using OpenSans

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[Aakash Desai [:aakashd]]

Bram noticed that OpenSans[1] is called out on the CSS we're using on Bootstrappified mozillians-dev, but the headings are showing helvetica/georgia. Not sure what's going on, but we should be using OpenSans. Is it installed onto the dev server?


[1]http://www-dev.allizom.org/b/en-US/sandstone/

Comment 1

[Aakash Desai [:aakashd]]

Attachment: bug screenshot

Comment 2

[Bram Pitoyo [:bram]]

I noticed that the page did call for the Open Sans family that are hosted on Google Webfonts Libary, and the stylesheet did specify it. But I see Lucida, which is the second font in that stack.

Comment 3

[Owen Coutts [:tallOwen]]

Lets wait on this until we have advice from visual designers. (The current fonts are a default set that was put together by a designer who knows far more about this stuff than I do :P)

Comment 4

[Aakash Desai [:aakashd]]

Hey Lee, can you help us out with figuring out the default set of fonts for sandstone. Owen needs some help here.

Comment 5

[Bram Pitoyo [:bram]]

I don’t do design as much as I do typography, so here’s my interpretation of the Sandstone guideline:
* Headings (h1–h6) and large texts should be Open Sans Light in these sizes: 108px, 72px, 48px, 32px, 28px and 24px
* Primary body text should remain Georgia, with a base size of 16px and line-height of 24 px
* Open Sans Book could be used for alternate body text type (ie. caption), but usually there’s no need to do this

Comment 6

[Bram Pitoyo [:bram]]

(In reply to Bram Pitoyo [:bram] from comment #5)
> * Open Sans Book could be used for alternate body text type (ie. caption),
> but usually there’s no need to do this

Alternate body text would include elements like the navigation menu, sidebar and footer.

Comment 7

[Lee Tom]

Sorry I'm so late to this thread. Bram pretty much summed up the guidelines well here. There are often going to be scenarios where these guides can bend a little, but it's definitely a good place to start. Thanks Bram!

Comment 8

[[github robot]]

Commit pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/7a5274eb4089d292952184d3919c6937bbe9c815
* Fixed bug #737438
* Changed the navbar font to OpenSans
* Changed quote to double quote

Comment 9

[Aakash Desai [:aakashd]]

They sure do look pretty on dev! Congrats daker!

Comment 10

[Bram Pitoyo [:bram]]

Awesome! I know that lee had started mocking up

Comment 11

[Bram Pitoyo [:bram]]

Sorry for the incomplete reply. The complete one, below:

I know that lee had started mocking up the Registration flow pages, and this includes the new Mozillians home page that everyone will see. These pages would be a good thing to design next, after this bug is done.

Comment 12

[Matt Brandt [:mbrandt]]

QA verified on dev