Skip to content
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

Fringe cases with lists and html #43

Open
fenugeek opened this issue May 2, 2019 · 1 comment
Open

Fringe cases with lists and html #43

fenugeek opened this issue May 2, 2019 · 1 comment

Comments

@fenugeek
Copy link

fenugeek commented May 2, 2019

I love this add-on! I'm finding a few inconsistencies in the list handing still, though.
A few edge-cases I've found with lists still not nesting/behaving quite right in html. All three cases here were found when converting real docs.

First case (two lists next to each other, eg list one is numbered, list 2 is bullet points))
1. List 1 item 1
* list 2 item 1
Generates to the below (misses the fact it's a second list)
<ol>
<li>List 1 item 1
<li>List 2 item 1

Second case (a blank line between two lists):
1. List 1 item 1

* list 2 item 1
Generates to the below (indents it up another level; doesn't ever close the </ol>)
<ol>
<li>List 1 item 1
<ul>
<li>List 2 item 1

Third case (indented trailing text after the list; </li> should be before the text, not after it)
1. List 1 item 1
Indented text
Generates to the below. The example isn't clear in the markup, but 'indented text' is indented by whitespace, or the margin is different - anything other than being up against the left hand margin.
<ol>
<li>List 1 item 1
<p>Indented text</p>
</li>
</ol>

Thanks for this awesome add-on!

@michael-ts
Copy link

I just want to add on to this that plain idention doesn't appear to work, either.

Example: this is what it looks like in Google Docs:
Selection_081

This is the rendered HTML:

<p>
Indention Test.  The next line is indented one stop.
</p>
<p>

    This is the intented text
</p>
<p>
Back to not indented
</p>

It appears that it's trying to render the indent with a bunch of extra space - obviously this won't work in HTML as all the extra space will be ignored.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants