search

ggez / ggez.github.io

ggez website
http://ggez.rs/
3 stars 4 forks source link

Ensure sans-serif fonts are used #2

Closed 17cupsofcoffee closed 7 years ago

17cupsofcoffee commented 7 years ago

Noticed while looking at the site on Windows that all the fonts were displaying with serifs:

screenshot of the site with serif text ):

Based on the selection of fonts listed in the CSS, I'm pretty sure this wasn't what you were going for! This PR adds sans-serif on to the end of the font stack so that it'll always fall back to a sans-serif font if the listed ones aren't found.

I've done the bare minimum fix here, but it's worth asking - was your intent to use the system/default fonts where applicable? If so, it might be worth switching to something like this to ensure things work well on all platforms. I can update this PR if that's something you'd be interested in!

icefoxen commented 7 years ago

Good question; I don't know what svenstaro's original intent was, so I'll add this patch but leave this PR open for further comment.

svenstaro commented 7 years ago

Oh sorry totally missed this. I'm currently at a conference. Will handle this later!