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

Fixed svg end tag token #124220

Closed
wants to merge 1 commit into from
Closed

Conversation

johnsoncodehk
Copy link
Contributor

This PR fixes #124215.

The problem is \\s* can't catch new line character. Split (?i)(</)(\\2)\\s*(>) to two patterns by \\s*(>), (?i)(</)(svg)\\b to fixed.

@aeschli
Copy link
Contributor

aeschli commented May 20, 2021

@johnsoncodehk Thanks a lot for looking into this. Please see the file header:

This file has been converted from https://github.com/textmate/html.tmbundle/blob/master/Syntaxes/HTML.plist

Can you make the PR against https://github.com/textmate/html.tmbundle ? Once accepted there we run our update script to bring in the version from there.

@johnsoncodehk
Copy link
Contributor Author

Sorry I am not familiar with this repo, and currently I don't have time to check it. But I have shared this resolve to textmate/html.tmbundle#101.

@github-actions github-actions bot locked and limited conversation to collaborators Jul 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

svg tag in HTML mode incorrectly marked invalid when closing tag spans more than one line
2 participants