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

Fix highlight tables #800

Merged
merged 1 commit into from Jun 18, 2018

Conversation

Projects
None yet
2 participants
@facelessuser
Copy link
Contributor

facelessuser commented Jun 18, 2018

When codehilite is disabled, or if pymdownx.highlight is used instead of codehilite, the default class "highlight" is used instead of "codehilite". This also applies to the number table format, and will use "highlighttable" instead of "codehilitetable".

Fix highlight tables
When codehilite is disabled, or if pymdownx.highlight is used instead of codehilite, the default class "highlight" is used instead of "codehilite".  This also applies to the number table format that, and will use "highlighttable" instead of "codehilitetable".
@squidfunk

This comment has been minimized.

Copy link
Owner

squidfunk commented Jun 18, 2018

Awesome, thanks for fixing. Really hard to keep track of those many combinations of plugins.

@squidfunk squidfunk merged commit 31e5612 into squidfunk:master Jun 18, 2018

2 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@squidfunk

This comment has been minimized.

Copy link
Owner

squidfunk commented Jun 18, 2018

Released as part of 2.9.1

@facelessuser

This comment has been minimized.

Copy link
Contributor Author

facelessuser commented Jun 18, 2018

Awesome, thanks for fixing. Really hard to keep track of those many combinations of plugins.

Yeah. I'm basically dropping use of Codehilite in my documents (I never use indented code blocks the way Codehilite wants me to use them), and it is kind of weird to set my class to the name of a plugin that I'm not using 🙂. I've been meaning to patch this for a while, but finally got around to it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.