A lighter, smarter, font declaration #3360

Merged
merged 1 commit into from Feb 2, 2015

Conversation

Projects
None yet
4 participants
@nternetinspired
Contributor

nternetinspired commented Jan 26, 2015

This PR introduces a new $base-font-weight variable, as we already have a var for other base font values, and then declares the <body> font properties with a single declaration in place of four.

Easier to configure, less css, same visual output.

@XhmikosR

This comment has been minimized.

Show comment
Hide comment
@XhmikosR

XhmikosR Jan 29, 2015

Contributor

👍 for using the font shorthand property.

You should squash those two patches.

Contributor

XhmikosR commented Jan 29, 2015

👍 for using the font shorthand property.

You should squash those two patches.

A lighter font declaration
Removes unnecessary Sass

Thanks @kleinfreund :)
@nternetinspired

This comment has been minimized.

Show comment
Hide comment
@nternetinspired

nternetinspired Feb 2, 2015

Contributor

👍

Squashed. Thanks @XhmikosR

Contributor

nternetinspired commented Feb 2, 2015

👍

Squashed. Thanks @XhmikosR

@parkr parkr merged commit 2bec317 into jekyll:master Feb 2, 2015

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

parkr added a commit that referenced this pull request Feb 2, 2015

tkrotoff added a commit to tkrotoff/osteo15.com that referenced this pull request Feb 16, 2015

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.