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
fix: remove table formatting from github gists #416
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Just a small change request.
src/sass/_markdown.scss
Outdated
width: 100%; | ||
text-align: left; | ||
.table-wrap { | ||
table:not(.lntable) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to do the opposite and use table:not(.lntable):not(.gist .highlight)
.
Fixed CI and added a Gist usage example. Thanks again! |
When using the
gist
shortcode to include a code block into the site, the css code generated from the_markdown.scss
will cause the gist to include extra padding and bottom borders due to how thetable
css is laid out.This is the problematic line..
It seems that there's already a structure in place to solve this issue, as there's already a utility method that wraps all written
<table>
tags in<div="table-wrap">
tags. So this PR moved the scss to only target those tables.This results in the correct look: