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

Textmate: fix Razor expression classification inside of HTML tags #7051

Merged
merged 4 commits into from Apr 16, 2024
Merged

Textmate: fix Razor expression classification inside of HTML tags #7051

merged 4 commits into from Apr 16, 2024

Conversation

jordi1215
Copy link
Member

@jordi1215 jordi1215 commented Apr 15, 2024 •
edited

  • added textmate grammar to both razor and html (test) files so that we can correctly label razor components that are within html tags. We will need to update the html grammar used in razor as well
  • updated tests to reflect the desire behavior, note that we changed some of the classification from existing tests as they were not correctly labeling razor grammar in html tags either

Before

before_vs_code

After

after_vs_code

@jordi1215
fix html-razor textmate
c05fb43 
- added textmate grammar to both razor and html (test) files so that we can correctly label razor components that are within html tags. We will need to update the html grammar used in razor as well
- updated tests to reflect the desire behavior, note that we changed some of the classification from existing tests as they were not correctly labeling razor grammar in html tags either
@jordi1215
fix html-razor textmate
d2b40e5 
- added textmate grammar to both razor and html (test) files so that we can correctly label razor components that are within html tags. We will need to update the html grammar used in razor as well
- updated tests to reflect the desire behavior, note that we changed some of the classification from existing tests as they were not correctly labeling razor grammar in html tags either
@jordi1215
Merge branch 'dev/jordi1215/fix-html-textmate' of https://github.com/…
a39a9f1 
…dotnet/vscode-csharp into dev/jordi1215/fix-html-textmate
@jordi1215 jordi1215 requested review from a team as code owners April 15, 2024 21:19
davidwengier
davidwengier approved these changes Apr 15, 2024
View reviewed changes
Copy link
Contributor

@davidwengier davidwengier left a comment

Choose a reason for hiding this comment

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

As discussed, I think this is good as a workaround for VS, but would prefer a better solution where semantic classification is considered.

src/razor/syntaxes/aspnetcorerazor.tmLanguage.json Outdated Show resolved Hide resolved
@jordi1215
update the textmate grammar clause name
5f9dfe1 
we updated the a textmate grammar clause name in razor to be more specific
@jordi1215 jordi1215 mentioned this pull request Apr 16, 2024
Merged
@jordi1215 jordi1215 merged commit 7166bf7 into main Apr 16, 2024
13 checks passed
@jordi1215 jordi1215 deleted the dev/jordi1215/fix-html-textmate branch April 16, 2024 17:51
@jordi1215 jordi1215 mentioned this pull request Apr 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidwengier davidwengier davidwengier approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jordi1215 @davidwengier