Skip to content
This repository has been archived by the owner on Aug 30, 2018. It is now read-only.

Header tags and em Sass function #183

Merged
merged 2 commits into from
Aug 15, 2014
Merged

Header tags and em Sass function #183

merged 2 commits into from
Aug 15, 2014

Conversation

cshold
Copy link
Contributor

@cshold cshold commented Aug 15, 2014

  • Added a function to convert pixel font sizes to em automatically
    • @mpiotrowicz, used a slight variation of the one you pointed me to. Uses em(20px) rather than just em(20)
    • I think it makes it a bit more clear to understand what is happening with the unit attached
  • Cleaned up use of h1 and h2 page title tags (as per recent comment on Schema article #90)
  • Minor cleaning of padding/margins to use gutter rather than ems

@jholl @stevebosworth

@mpiotrowicz
Copy link
Contributor

works for me!

cshold added a commit that referenced this pull request Aug 15, 2014
Header tags and em Sass function
@cshold cshold merged commit fb30752 into master Aug 15, 2014
@cshold cshold deleted the header-tags branch August 15, 2014 15:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants