asciidoctor / kramdown-asciidoc

A kramdown extension for converting Markdown documents to AsciiDoc.
Other
207 stars 19 forks source link

Links with pipe in name are converted to tables #117

Closed ViToni closed 7 months ago

ViToni commented 7 months ago

This Markdown has a link with a pipe in its name. It seems that the parser treats it as a column seprator and by that changes the name of the link and tries to format the link as a table:

Markdown:

# Test Markdown

* Links
  * [Asciidoctor Documentation Site](https://docs.asciidoctor.org/asciidoc/latest/macros/)
  * [Links | Asciidoctor Docs](https://docs.asciidoctor.org/asciidoc/latest/macros/links/)
  * [asciidoctor/kramdown-asciidoc: A kramdown extension for converting Markdown documents to AsciiDoc.](https://github.com/asciidoctor/kramdown-asciidoc)

Actual result:

= Test Markdown

* Links
 ** https://docs.asciidoctor.org/asciidoc/latest/macros/[Asciidoctor Documentation Site]
 ** {blank}
+
[cols=2*]
|===
| [Links
| Asciidoctor Docs](https://docs.asciidoctor.org/asciidoc/latest/macros/links/)
|===

 ** https://github.com/asciidoctor/kramdown-asciidoc[asciidoctor/kramdown-asciidoc: A kramdown extension for converting Markdown documents to AsciiDoc.]

Expected result:

= Test Markdown

* Links
 ** https://docs.asciidoctor.org/asciidoc/latest/macros/[Asciidoctor Documentation Site]
 ** https://docs.asciidoctor.org/asciidoc/latest/macros/links/[Links | Asciidoctor Docs]
 ** https://github.com/asciidoctor/kramdown-asciidoc[asciidoctor/kramdown-asciidoc: A kramdown extension for converting Markdown documents to AsciiDoc.]
ViToni commented 7 months ago

Seems related to https://github.com/gettalong/kramdown/issues/431. So it's not an actual bug, but the pipe symbol needs to be escaped as \| to work as expected.