Skip to content

Commit

Permalink
Set <article> on narrative and correct CSS
Browse files Browse the repository at this point in the history
This also adds `.narrative` alongside the existing (and more brittle)
`div.narrative` selector, to ensure fuller backward compatibility. In
practice, eliminating `div.narrative` would probably be fine. But
preserving it won't hurt anything, either.
  • Loading branch information
karlstolley committed Sep 18, 2019
1 parent 63a8990 commit 0aa0d92
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 2 additions & 2 deletions _layouts/narrative.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
layout: default
---

<div class="narrative">
<article class="narrative">
<h1 class="text-title">{{ page.title }}</h1>
<p class="citation"> by {{ page.author }}</p>
{{ content }}
</div>
</article>
3 changes: 2 additions & 1 deletion _sass/_ed.scss
Original file line number Diff line number Diff line change
Expand Up @@ -664,7 +664,8 @@ p.small {
}

div.poem,
div.narrative {
div.narrative,
.narrative {
margin-bottom: 5rem;
}

Expand Down

0 comments on commit 0aa0d92

Please sign in to comment.