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.

Sign up for GitHub

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

Jump to bottom

Remove optional open/closing tags #29

Open
maltsev opened this issue Mar 5, 2016 · 3 comments
Open

Remove optional open/closing tags #29

maltsev opened this issue Mar 5, 2016 · 3 comments
Labels
enhancement new module

Comments

@maltsev
Copy link
Member

maltsev commented Mar 5, 2016

@stevenvachon:

  1. Remove optional closing tags such as </html>, </head>, </body>, </option>, </p>, </thead>, </tbody>, </tfoot>, </tr>
  2. remove optional opening tags such as <html>, <head>, <body>
@wooorm
Copy link

wooorm commented Jan 4, 2019

Here’s what I did for opening and closing tags.

@SukkaW
Copy link
Contributor

SukkaW commented Oct 11, 2020

@maltsev considering the feature will result in invalid HTML, will posthtml be compatible with it?

@maltsev maltsev mentioned this issue Oct 11, 2020
Open
@maltsev
Copy link
Member Author

maltsev commented Oct 11, 2020

considering the feature will result in invalid HTML, will posthtml be compatible with it?

I'm not sure. We should test it.

On a more general note, it raises a more important question about PostHTML limitations. I created #99 for discussing that.

@SukkaW SukkaW mentioned this issue Oct 18, 2020
Merged
@SukkaW SukkaW mentioned this issue Nov 24, 2020
Closed
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement new module
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@maltsev @wooorm @SukkaW and others