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

Enable syntax highlighting for vue files using <script setup> syntax #2918

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

charlesbrandt
Copy link

Addresses the syntax highlighting issue raised in #2381

Comment on lines 59 to 63
- default:
start: "<script[ ]+setup>"
end: "</script>"
rules:
- include: "javascript"

This comment was marked as resolved.

Comment on lines 65 to 69
- default:
start: "<script[ ]+setup[ ]+lang=(\"ts\"|'ts')>"
end: "</script>"
rules:
- include: "typescript"

This comment was marked as resolved.

Comment on lines 78 to 83
- default:
start: "<style[ ]+scoped.*?>"
end: "</style.*?>"
limit-group: symbol.tag
rules:
- include: "css"

This comment was marked as resolved.

@charlesbrandt
Copy link
Author

Thank you for the feedback. I attempted to address your suggestions in my latest commit. I'm also simplifying the <template> section by including the standard html rules. This seems to work better.



Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You added unintended white spaces?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @JoeKar . Got it

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