uccser / verto

Verto is an extension of the Python Markdown package, which allows authors to include complex HTML elements with simple text tags in their Markdown files.
Other
2 stars 1 forks source link

[Bug]: Add test and support for links with nested ()s #657

Open jimbonothing64 opened 3 months ago

jimbonothing64 commented 3 months ago

Description of the bug

Bug

Links with ()s do not parse correctly. See https://github.com/uccser/cs-field-guide/issues/2565 (Thanks to @mckzm for spotting this and working on a fix!)

Proposed fix

Background

In https://github.com/uccser/cs-field-guide/pull/2566, the problem links were enclosed in <>, as per Common mark spec. But pyton-markdown uses John Gruber's Spec, which does not support enclosing links in <>.

Using a newer version of python-markdown should fix these problem links, and still conformant to the John Gruber's Spec.

See blame for date fix was added to python-markdown: https://github.com/Python-Markdown/markdown/blame/7c382f6c5d438d6d6661f842b03f15dbd6e42a2d/markdown/inlinepatterns.py#L449

Steps To Reproduce

Include [Entropy and information theory](https://en.wikipedia.org/wiki/Entropy_(information_theory)) in a template a markdown file and then render it using verto.

What device/software are you seeing this problem on?

No response

Additional Information

No response

Code of Conduct

jimbonothing64 commented 3 months ago

Updating to markdown 3.0 does not fix parsing for [Entropy and information theory](https://en.wikipedia.org/wiki/Entropy_(information_theory)). Updating further than 3.0 causes more issues; no immediate action.