Skip to content

Having only one list item in the root that is not last creates invalid html #24

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

Merged
merged 9 commits into from
May 21, 2013

Conversation

jlward
Copy link
Contributor

@jlward jlward commented May 16, 2013

No description provided.

@ghost ghost assigned jlward May 16, 2013
@@ -460,10 +460,29 @@ def parse_list(self, el, text):
return self.parse_table_cell_contents(el, parsed)
return parsed

def _build_list(self, el, text):
# Get the list style for the pending list.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And this actually makes the code cleaner, anyway. Nice

@winhamwr
Copy link
Contributor

Looks good if you add a comment there. It kind of makes me think that is_last_list_item_in_root either might not be a great name or that another function should exist, but a comment will probably cover it.

Jason Ward added 2 commits May 21, 2013 10:42
jlward added a commit that referenced this pull request May 21, 2013
Having only one list item in the root that is not last creates invalid html
@jlward jlward merged commit cdd0add into master May 21, 2013
jlward pushed a commit that referenced this pull request Mar 14, 2014
…ml. Also updated the justification test case to generate valid html as well.
jlward pushed a commit that referenced this pull request Mar 14, 2014
jlward pushed a commit that referenced this pull request Mar 14, 2014
jlward pushed a commit that referenced this pull request Mar 14, 2014
jlward added a commit that referenced this pull request Mar 14, 2014
Having only one list item in the root that is not last creates invalid html
jlward pushed a commit that referenced this pull request Mar 14, 2014
…ml. Also updated the justification test case to generate valid html as well.
jlward pushed a commit that referenced this pull request Mar 14, 2014
jlward pushed a commit that referenced this pull request Mar 14, 2014
jlward pushed a commit that referenced this pull request Mar 14, 2014
jlward added a commit that referenced this pull request Mar 14, 2014
Having only one list item in the root that is not last creates invalid html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants