Skip to content

Commit

Permalink
Really fix the padding across views this time
Browse files Browse the repository at this point in the history
* Moves padding away from `.post` and to `.content`
* Removes top margin from `.post` since padding is now on .content
* Streamlines responsive CSS to account for that
* Fixes inconsistent padding in related posts due to top margin on h2
* Drop the `.fourohfour` class since we don’t need it (as padding is on
the `.content` now)
  • Loading branch information
mdo committed Dec 28, 2013
1 parent 5534929 commit 44c86e0
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 24 deletions.
12 changes: 5 additions & 7 deletions 404.html
Original file line number Diff line number Diff line change
Expand Up @@ -44,13 +44,11 @@ <h1>Hyde</h1>
</header>

<div class="content container">
<div class="fourohfour">
<h1>404: Page not found</h1>
<p class="lead">Sorry, we've misplaced that URL or it's pointing to something that doesn't exist. <a href="/">Visit the home page</a> to see if you can find it manually. Thanks!</p>
<hr>
<h3>Heads up!</h3>
<p>This is a basic 404 page for use with GitHub Pages. It's source code is not generated by Jekyll.</p>
<div>
<h1>404: Page not found</h1>
<p class="lead">Sorry, we've misplaced that URL or it's pointing to something that doesn't exist. <a href="/">Visit the home page</a> to see if you can find it manually. Thanks!</p>
<hr>
<h3>Heads up!</h3>
<p>This is a basic 404 page for use with GitHub Pages. It's source code is not generated by Jekyll.</p>
</div>

</body>
Expand Down
28 changes: 11 additions & 17 deletions public/css/hyde.css
Original file line number Diff line number Diff line change
Expand Up @@ -299,12 +299,19 @@ a:focus {
* with a 25%-wide `.masthead`.
*/

.content {
padding: 40px 20px;
}

/* Center container in available real estate */
@media (min-width: 990px) {
.content {
padding: 60px 0;
}
.container {
width: 55%;
margin-left: 35%;
margin-right: 10%;
width: 55%;
}
}

Expand All @@ -317,12 +324,11 @@ a:focus {

/* Single post */
.post {
margin: 40px 20px;
margin-bottom: 40px;
}
@media (min-width: 990px) {
/* Increase space between posts */
.post {
margin: 60px 0; /* Increase top/bottom, remove from sides */
margin-bottom: 60px;
}
}

Expand All @@ -335,7 +341,7 @@ a:focus {

/* Related posts */
.related {
padding-top: 40px;
padding-top: 20px;
padding-bottom: 40px;
border-top: 1px solid #eee;
}
Expand All @@ -359,18 +365,6 @@ a:focus {
}


/*
* Error page
*
* For now, just the 404 (supported by Jekyll with the custom 404.html view).
*/

.fourohfour {
margin-top: 40px;
margin-bottom: 40px;
}


/*
* Pagination
*
Expand Down

0 comments on commit 44c86e0

Please sign in to comment.