search

AnWeber / atom-htmltidy

Atom Package to tidy up HTML with HTML Tidy 5
MIT License
2 stars 1 forks source link

incorrectly wrapping list items #5

Closed garretwilson closed 8 years ago

garretwilson commented 8 years ago

I have the following options:

{"input-xml" : true, "output-xml" : true, "indent" : true, "indent-with-tabs" : true, "wrap" : 0}

I have the following input:

<ul>
  <li><code>args4j:args4j:2.32</code></li>
</ul>

atom-htmltidy wraps that to:

    <ul>
        <li>
            <code>args4j:args4j:2.32</code>
        </li>
    </ul>

Why the newlines inside <li>? How do I get rid of them? <code> is an inline element --- there is no need for wrapping.

AnWeber commented 8 years ago

the atom plugin is just a wrapper for Tidy-HTML5, Documentation.

In my opinion the problem is the option "output-xml". If this option is set html-tidy interprets all tags as block level element tags. In my project I don't use this option anymore (htmltidy.txt)

If I couldn't help, please open an issue here.

Thanks

garretwilson commented 8 years ago

In my opinion the problem is the option "output-xml"

You didn't try it did you? You made an assumption about the problem, didn't test the provided source, and closed the issue.

AnWeber commented 8 years ago

Yes, because it is an known issue at tidy-html

garretwilson commented 8 years ago

But I see it still occurring without the output-xml option. That's why I asked if you tested it.

On January 8, 2016 2:37:40 PM GMT-02:00, Andreas Weber notifications@github.com wrote:

Yes, because it is an issue at tidy-html

Reply to this email directly or view it on GitHub: https://github.com/AnWeber/atom-htmltidy/issues/5#issuecomment-170049773

AnWeber commented 8 years ago

Yes, I tested it. I`ve got the same output on the command line by using tidy.exe. Using the option 'new-inline-tags' didn't solve the problem. Please open an issue here.

Thanks.