search

googlearchive / py-gfm

This repository is unmaintained. Please see Zopieux/py-gfm for the new canonical repository.
https://github.com/Zopieux/py-gfm
BSD 3-Clause "New" or "Revised" License
73 stars 30 forks source link

GitHub supports ul indenting using one or two spaces but this module requires the Markdown specification of 4 spaces. #23

Open chr15m opened 9 years ago

chr15m commented 9 years ago

I think it would make sense to ignore the specification and do what GitHub allows since the name of the module is "GitHub Flavoured Markdown".

Test markdown to replicate this bug:

 * First item.
   * First indented item.
   * Second indented item.
 * Second item.
 * Third item.

The sub-list indenting works correctly on GitHub and with the original markdown.pl implementation, but does not work with py-gfm or several other markdown renderers.

chr15m commented 9 years ago

For anybody finding this issue, the python module markdown2 renders lists consistently with GitHub and the original markdown.pl, against the spec.