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

Ensure empty inline svg and math tags are serialized as void tags #259

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ohader
Copy link
Contributor

@ohader ohader commented Mar 6, 2025

Before this change, e.g. <svg></svg> has been incorrectly serialized as <svg /></svg>.
This change ensures, that the result is a proper (self-closing) void tag <svg />.

@ohader
Copy link
Contributor Author

ohader commented Mar 6, 2025

Rebase on PR #260 once merged for addressing php-cs-fixer remarks.

@xabbuh
Copy link
Collaborator

xabbuh commented Mar 7, 2025

@ohader #260 has been merged

Before this change, e.g. `<svg></svg>` has been incorrectly
serialized as `<svg /></svg>`. This change ensures, that the
result is a proper (self-closing) void tag `<svg />`.
@ohader ohader force-pushed the void-svg-math-tags branch from 35c9c84 to c97a75a Compare March 7, 2025 10:17
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.

None yet

2 participants