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

Restore config for removing self closing tags in attributes #765

Closed
wants to merge 1 commit into from

Conversation

edg2s
Copy link
Contributor

@edg2s edg2s commented Feb 4, 2023

Fixes #761

@cure53
Copy link
Owner

cure53 commented Feb 6, 2023

This has conflicts, please resolve before we can merge @edg2s

@cure53
Copy link
Owner

cure53 commented Feb 6, 2023

It's not clear to me what branch this change should land in. Is MSIE support still needed? If so, we need this in 2.x and 3.x (or main).

@AndreVirtimo
Copy link

For me MSIE support is not needed. This browser is EOL since last year.

@cure53
Copy link
Owner

cure53 commented Feb 6, 2023

Same here, we want to get rid of it asap and more on with 3.x 😄 But not sure about Wikipedia @edg2s

@edg2s
Copy link
Contributor Author

edg2s commented Feb 6, 2023

We haven't dropped support for IE11 just yet, so would prefer to do this is in 2.x.

@cure53
Copy link
Owner

cure53 commented Feb 7, 2023

Closing this for now in the hopes that the recent changes to 2.x do the trick 😃 Please reopen in case I overlooked something.

@cure53 cure53 closed this Feb 7, 2023
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

Successfully merging this pull request may close these issues.

[Bug] HTML encoded self closing tags in attribute values leads to the removal of the attribute
3 participants