Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor styling error with title-only admonition blocks #616

Closed
ZaLiTHkA opened this issue Nov 27, 2017 · 4 comments
Closed

Minor styling error with title-only admonition blocks #616

ZaLiTHkA opened this issue Nov 27, 2017 · 4 comments
Assignees
Labels

Comments

@ZaLiTHkA
Copy link

@ZaLiTHkA ZaLiTHkA commented Nov 27, 2017

Description

When using admonition as only the title and no body..

!!! info "hello world"

..there's some bottom padding that pushes out the wrapping container a bit, leaving a white bar along the bottom of the coloured section of the admonition block.

Expected behavior

I expect everything to be pixel-perfect, with no unwanted padding or blurry corners! Lol. 🖖

Actual behavior

As per screenshot:

2017-11-27 15_04_24-without-fix

Steps to reproduce the bug

n/a

Package versions

  • Python: 2.7.13
  • MkDocs: 0.17.1
  • Material: 2.2.0

Project configuration

n/a

System information

  • OS: Windows 10 x64
  • Browser: Google Chrome 62.0.3202.94
@ZaLiTHkA
Copy link
Author

@ZaLiTHkA ZaLiTHkA commented Nov 27, 2017

Not sure if this is the most elegant solution, there could be better ways to handle it, but in the meantime I'm getting around this with the following custom CSS:

.md-typeset .admonition, .md-typeset details {
  padding-bottom: 0;
}

html .md-typeset .admonition :last-child, html .md-typeset details :last-child {
  padding-bottom: 1rem;
}
@squidfunk squidfunk added the bug label Nov 28, 2017
@squidfunk squidfunk self-assigned this Nov 28, 2017
@squidfunk
Copy link
Owner

@squidfunk squidfunk commented Dec 2, 2017

Fixed in #621.

@squidfunk
Copy link
Owner

@squidfunk squidfunk commented Dec 2, 2017

Released as 2.2.1.

@squidfunk squidfunk closed this Dec 2, 2017
@ZaLiTHkA
Copy link
Author

@ZaLiTHkA ZaLiTHkA commented Dec 14, 2017

Thanks for the fix. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants