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

tactic#interval_cases: syntax highlighting not working #86

Closed
bryangingechen opened this issue Oct 2, 2020 · 2 comments
Closed

tactic#interval_cases: syntax highlighting not working #86

bryangingechen opened this issue Oct 2, 2020 · 2 comments

Comments

@bryangingechen
Copy link
Collaborator

See https://leanprover-community.github.io/mathlib_docs/tactics.html#interval_cases

Not sure what's going on here, since the highlighting here works, and the relevant parts of the docstring markdown look the same to me: compare https://github.com/leanprover-community/mathlib/blob/d41f386/src/tactic/interval_cases.lean#L213 and line 249.

@gebner
Copy link
Member

gebner commented Oct 2, 2020

The obvious difference is that one is a docstring for a declaration, while the other is a docstring for a user command.

I would conjecture that the preprocessing step which replaces ``` by ```lean is only used for declaration docstrings.

As a fix, we could default to lean when highlighting markdown code blocks. This would require us to switch to the mistletoe renderer (used by the website), because it's hard to change the syntax highlighting in the markdown2 renderer.

@robertylewis
Copy link
Member

Fixed by #110

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

No branches or pull requests

3 participants