spacing in admonitions

[choldgraf]

This adds spacing for admonitions.

Demo it here: https://370-130237506-gh.circle-artifacts.com/0/html/demo/demo.html#admonitions

closes #201

[choldgraf]

OK I'm gonna merge this one in, since this is basically the implementation of the preview that @jorisvandenbossche +1'ed in #201 - I think it'll be easier to decide if we like the design after seeing it in action for a bit!

[jorisvandenbossche]

Thanks!

[pradyunsg]

Hmm... Comparing to https://squidfunk.github.io/mkdocs-material/reference/admonitions/, given that both follow the same style, I think I prefer the full-width variant from mkdocs-material. Would it be OK to drop the width: 95% added here?

[jorisvandenbossche]

Personally I would be fine with that, not sure if @choldgraf had a specific reason to add the padding

[choldgraf]

I don't think I have a strong opinion here, was just trying to find ways to differentiate admonitions from the rest of the text because it felt visually cluttered to me, but happy to see what it looks like at 100

[pradyunsg]

Hmm... Looks like I'll have to unbreak my npm setup to work on this project.

[jorisvandenbossche]

Hmm... Looks like I'll have to unbreak my npm setup to work on this project.

What do you mean exactly?
(insights in / feedback on how smooth (or not) the contributing process goes is very welcome, as we know there are some potential roadblocks ..)

[pradyunsg]

I'd borked my nvm / nodejs / brew installations a little while back and never fixed that trash fire because it was easier to step away and use Python-only tools.

I don't think the development workflow here is difficult by any means. It's just that I made a mess that I have to clean up. 🙃