Skip to content

readthedocs+recommonmark still does not get code highlighting #1859

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

Closed
kozo2 opened this issue Dec 11, 2015 · 6 comments
Closed

readthedocs+recommonmark still does not get code highlighting #1859

kozo2 opened this issue Dec 11, 2015 · 6 comments
Labels
Bug A bug Needed: more information A reply from issue author is required

Comments

@kozo2
Copy link

kozo2 commented Dec 11, 2015

This is already issued in #1485
but readthedocs+recommonmark still does not get code highlighting.

Please compare
http://markdown-code-highlight.readthedocs.org/en/latest/markdown.html
with
http://readthedocs-sphinx-markdown-issue.readthedocs.org/en/latest/rst.html

The code repo is
https://github.com/kozo2/readthedocs-sphinx-markdown-issue

@agjohnson
Copy link
Contributor

Yeah, that looks like it should work, seems there is a regression somewhere.

@agjohnson agjohnson added the Bug A bug label Dec 14, 2015
@pardahlman
Copy link

I've noticed this, too. Issue #1485 is closed but the example links @kozo2 posted in this issue are rendered without syntax highlight. Any news for this issue?

@tigerhawkvok
Copy link

On this bug, it's good to note that the collapsable navigation doesn't show up for the markdown, either -- which is to say,

# Foo
## Bar

Doesn't nest "Foo" under "Bar", but shows them as siblings.

@agjohnson
Copy link
Contributor

@tigerhawkvok your issue is not related to this issue, nor can I reproduce it. I assume this is because you are using mkdocs, and not Sphinx + recommonmark.

@stsewd
Copy link
Member

stsewd commented May 17, 2018

I'm not sure if this issue is still relevant, I can see both html outputs are the same. Also, we need to use the rtdtheme>0.3 to test this.

@stsewd stsewd added the Needed: more information A reply from issue author is required label May 17, 2018
@stsewd
Copy link
Member

stsewd commented May 17, 2018

I tested this on my local instance with the latest rtd-theme, it works as expected

screenshot-2018-5-17 rst header test test documentation

screenshot-2018-5-17 2nd level rst header test test documentation

Feel free to reopen this issue if I misunderstand something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug Needed: more information A reply from issue author is required
Projects
None yet
Development

No branches or pull requests

5 participants