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

Support in Markdown documents #114

Closed
harrytodorov opened this issue Apr 9, 2021 · 4 comments
Closed

Support in Markdown documents #114

harrytodorov opened this issue Apr 9, 2021 · 4 comments

Comments

@harrytodorov
Copy link

Hello kind developers,

Recently I had the task of integrating your extension in the docs of one of our products. I've liked the simplicity and visual reconfigurability of your extension.

However, our documentation is currently mostly written using CommonMark (through the recommonmark extension). That made the integration of your extension quite tedious and polluted our docs. We had to use special code blocks (for evaluating ReStructuredText) for each paragraph, where there was a term needing a tooltip:

```eval_rst
... :hoverxref:`some term <TERM_SOME_TERM>`
```

One of the main drawbacks of this is that you need to manually rewrite existing Markdown code to ReStructuredText in the paragraphs tooltips needed to be added.

In turn, this forced us to heavily reduce the number of places where would like to add tooltips. To my knowledge, there is also no option of inlining ReStructuredText in Markdown (that would have been a good compromise).

So the bottom line: from the perspective of our project, adding Markdown support would heavily increase the ease (and joy) of using your extension.

@stsewd
Copy link
Member

stsewd commented Apr 15, 2021

Hi, we recommend using myst instead of recommonmark if you want to author your docs using markdown https://myst-parser.readthedocs.io/en/latest/using/intro.html. MySt maps more nicely to the rst syntax. There isn't much we can do from the extension side to make it more easy with recommonmark.

@stsewd stsewd closed this as completed Apr 15, 2021
@humitos
Copy link
Member

humitos commented Apr 16, 2021

Hi @harrytodorov! Take into account that recommonmark is dropping official support, please read: readthedocs/recommonmark#221

I don't think we will implement support for it in this extension for that reason. I'd encourage you to test MyST-parser as @stsewd suggested and let us know if this extension works properly with MyST. I'd be happy to make the adjustments required to make it work there if any.

@harrytodorov
Copy link
Author

Hi @humitos! Thanks for the update on the issue!

We were not aware of this change. We have made the choice back then to use the recommonmark extension following the official suggestion from Sphinx (see here).

We will now follow the development and if we transition to MySt, then we can try to integrate hoverxref once again.

@humitos
Copy link
Member

humitos commented Apr 18, 2021

Great! Feel free to open an issue here if you find any problem with this extension! Thanks!

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