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

Copied aspnetcorerazor.tmLanguage.json from dotnet/vscode-csharp #9943

Merged
merged 1 commit into from Feb 16, 2024

Conversation

jordi1215
Copy link
Member

@jordi1215 jordi1215 commented Feb 15, 2024

This pull request fixes #9882, where Textmate did not support the newly added directives: rendermode, preservewhitespace, and typeparam. The TextMate grammar for Razor is located in the dotnet/vscode-csharp repository, and a separate pull request (dotnet/vscode-csharp#6887) has been submitted with the fix and tests.

Summary of the changes

The Razor tmLanguage.json file was copied from the vscode-csharp repository to reflect the latest changes, including the addition of grammar support for rendermode, preservewhitespace, and typeparam.

Fixes:

The lack of coloring for the new directives using TextMate.

Before

before_visual_studio

After

after_visual_studio

- copied over the tmLanguage.json file for razor from the vscode-csharpe repo to reflect the newest change after adding grammar support for rendermode, preservewhitespace, and typeparam
@jordi1215 jordi1215 requested a review from a team as a code owner February 15, 2024 23:12
@jordi1215 jordi1215 enabled auto-merge (squash) February 15, 2024 23:22
@jordi1215 jordi1215 merged commit f01e110 into main Feb 16, 2024
12 checks passed
@jordi1215 jordi1215 deleted the dev/jordi1215/9882-add-grammar-for-new-directives branch February 16, 2024 00:01
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Feb 16, 2024
@jjonescz jjonescz modified the milestones: Next, 17.10 P2 Feb 27, 2024
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.

Blazor @rendermode directive colouring
3 participants