stitchfix / stitchfix.github.io

15 stars 4 forks source link

New post + typography fixes #18

Closed davetron5000 closed 10 years ago

davetron5000 commented 10 years ago

Humblekit's .p class added weird line-height to code samples, because Pygments uses that class for somethingn other than "be like a <p> tag". Also removed the letter spacing on the code, which I think looks awkward and unnecessarily eats up horizontal space, which is at a premium for code listings.

Before

stitch_fix_technology

After

stitch_fix_technology

Also changed how inline code is styled so that it doesn't bump up the line-height of lines it's used on:

Before

stitch_fix_technology

After

stitch_fix_technology

yorthehunter commented 10 years ago

I'm guessing there's going to be a little bit of merge conflict (probably in application.scss) so I can wait to merge my stuff and fix when this is ready to go

jonathandean commented 10 years ago

Nice type changes!

jonathandean commented 10 years ago

ship it :+1: