Skip to content
This repository has been archived by the owner on Dec 11, 2021. It is now read-only.

Commit

Permalink
Typography: Eliminated need of a mixin for p
Browse files Browse the repository at this point in the history
Typography: Eliminated need of mixin for p
  • Loading branch information
thejdeep committed Mar 4, 2016
1 parent 14a3d8f commit 01d2ebe
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 11 deletions.
2 changes: 0 additions & 2 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,3 @@ external/
icons/svg-min/
.sass-cache/
dist/
.DS_Store
demos/.DS_Store
7 changes: 0 additions & 7 deletions scss/atoms/typography/_mixins.scss
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,3 @@
line-height: 1;
text-transform: map-get($style,capitalization);
}

@mixin paragraph($style) {
text-align: map-get($style,textAlignment);
margin: em(map-get($style,margin)) 0;
line-height: 1.5;
font-size: em(map-get($style,fontSize));
}
7 changes: 5 additions & 2 deletions scss/atoms/typography/_typography.scss
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,10 @@ h6 {
}

p {
@include paragraph($p);
text-align: map-get($p,textAlignment);
margin: em(map-get($p,margin)) 0;
line-height: 1.5;
font-size: em(map-get($p,fontSize));
}

ul, ol {
Expand All @@ -57,7 +60,7 @@ ul ul,
ol ul,
ul ol,
ol ol {
margin-bottom: 0;
margin-bottom: 0;
}

/*
Expand Down

0 comments on commit 01d2ebe

Please sign in to comment.