Skip to content

Commit

Permalink
fixing cuddled-lists with a single list item
Browse files Browse the repository at this point in the history
  • Loading branch information
hmagdy committed Apr 3, 2018
1 parent 0465b42 commit 9452600
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 6 deletions.
9 changes: 7 additions & 2 deletions lib/markdown2.py
Expand Up @@ -2056,8 +2056,13 @@ def _form_paragraphs(self, text):
# text (issue 33). Note the `[-1]` is a quick way to
# consider numeric bullets (e.g. "1." and "2.") to be
# equal.
if (li and len(li.group(2)) <= 3 and li.group("next_marker")
and li.group("marker")[-1] == li.group("next_marker")[-1]):
if (li and len(li.group(2)) <= 3
and (
(li.group("next_marker") and li.group("marker")[-1] == li.group("next_marker")[-1])
or
li.group("next_marker") is None
)
):
start = li.start()
cuddled_list = self._do_lists(graf[start:]).rstrip("\n")
assert cuddled_list.startswith("<ul>") or cuddled_list.startswith("<ol>")
Expand Down
4 changes: 2 additions & 2 deletions test/tm-cases/cuddled_para_and_list.html
Expand Up @@ -24,9 +24,9 @@
</ol>

<p>Here is an asterisk,
* What do you think
* What do you think
about it?</p>

<p>I suggest using version
8. Oops, now this line is treated
8. Oops, now this line is treated
as a sub-list.</p>
4 changes: 2 additions & 2 deletions test/tm-cases/cuddled_para_and_list.text
Expand Up @@ -15,10 +15,10 @@ And these in order:
3. bullet3

Here is an asterisk,
* What do you think
* What do you think
about it?

I suggest using version
8. Oops, now this line is treated
8. Oops, now this line is treated
as a sub-list.

0 comments on commit 9452600

Please sign in to comment.