<hr/> tag looks weird with margin: 0

[ozkriff]

Minima's _base.scss file resets the margin of the <hr> tag to zero:

rust-gamedev.github.io/_sass/minima/_base.scss

Lines 4 to 7 in 8dab245

	body, h1, h2, h3, h4, h5, h6,
	p, blockquote, pre, hr,
	dl, dd, ol, ul, figure {
	margin: 0;

And it results in that Markdown's horizontal line ------ sticks to the next line uncomfortably close:

With this brute-force patch:

$ git diff _sass/
diff --git a/_sass/minima/_base.scss b/_sass/minima/_base.scss
index db7d5a1..cac4ba7 100644
--- a/_sass/minima/_base.scss
+++ b/_sass/minima/_base.scss
@@ -37,6 +37,7 @@ body {
 h1, h2, h3, h4, h5, h6,
 p, blockquote, pre,
 ul, ol, dl, figure,
+hr,
 %vertical-rhythm {
   margin-bottom: $spacing-unit / 2;
 }

it looks much better to me:

But I'm note sure if that's the correct way of doing it.

[ozkriff]

Also, something seems to be wrong with nested lists' margins:

[17cupsofcoffee]

The only downside to editing the minima files would be we'd have to diff them if we upgrade to a newer version or risk losing your changes - the alternative would be to add hr { margin-bottom: $spacing-unit / 2; } to extra.scss, but that would bump the filesize a little. Tradeoffs to both ways :)

The nested list issue is my bad - change this part of extra.scss:

li {
    // For consistency with other elements.
    margin-bottom: 15px;
}

To this:

li {
    // For consistency with other elements.
    margin: 15px 0;
}

Might be worth adding these tweaks (and any others) to the newsletter PR if they're in service of making the newsletter look better!