This repository has been archived by the owner on Mar 22, 2019. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In the midst of the redesign it seems that the container sizes for the page content and header/footer has become inconsistent.
The header footer (and most of the site) uses a 960px container with
1rem
of horizontal padding.However there are a few text containers (most noticeably the
.legal-wrapper
) which are MUCH smaller.This is because the
.legal-wrapper
relied onem
units.Since the base
font-size
of these areas were reduced to14px
then60em
is no longer the loved960px
but instead comes out to840px
.On top of this, the padding of the
.legal-wrapper
container also used3rem
left and right.This meant that the container was even smaller than the
840px
.The content width of the navbar vs
.legal-wrapper
is928px
to756px
after padding and everything is done.This PR brings back consistency in container sizes.