808loud / charlie-portfolio

http://chrlieco.com
0 stars 0 forks source link

Line spacing for the header "Consultant for Hire" #62

Closed 808loud closed 8 years ago

808loud commented 8 years ago

@charlydsgn how do you want it changed?

808loud commented 8 years ago

Drop the text-shadow

808loud commented 8 years ago

Line-height 1.4

808loud commented 8 years ago

Print modal headlines has a long headline

808loud commented 8 years ago

Dash should line up in the middle of the whitespace of the content column.

Line-height 1 should do it, but apply only to the outline-headling class

808loud commented 8 years ago

Using line-height of 1 and margin-bottom of 0.4em to equate to a line-height of 1.4 while preserving the vertical position as shown in the design files. It's also a cleaner solution for calculating the height of the element.