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

languages: add mdx to markdown filetypes #11122

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

jzelinskie
Copy link
Contributor

mdx - markdown with jsx support is becoming quite popular for some react-based javascript frameworks
the standard markdown highlighting works fine

@the-mikedavis the-mikedavis added the A-language-support Area: Support for programming/text languages label Jul 10, 2024
@theduke
Copy link

theduke commented Jul 11, 2024

Would be nice to have proper support that can highlight both markdown and Javascript.

@jzelinskie
Copy link
Contributor Author

Would be nice to have proper support that can highlight both markdown and Javascript.

this is a great point. here's an example of the javascript imports not highlighting:
Screenshot 2024-07-11 at 10 54 13 PM

@jzelinskie
Copy link
Contributor Author

This appears to be blocked on upstream tree-sitter-grammars/tree-sitter-markdown#81

@pascalkuthe
Copy link
Member

seems like this needs a seperate mdx grammar to be solved which doens't exist yet...

tree-sitter-grammars/tree-sitter-markdown#137 (comment)

@pascalkuthe
Copy link
Member

I will merge this for now since markdown is quite close for this file format and this cannot be resolved upstream or on our side

@pascalkuthe pascalkuthe merged commit 5e945c3 into helix-editor:master Jul 25, 2024
6 checks passed
@jzelinskie jzelinskie deleted the mdx branch July 30, 2024 20:54
SofusA pushed a commit to SofusA/helix-pull-diagnostics that referenced this pull request Aug 4, 2024
mxxntype pushed a commit to mxxntype/helix that referenced this pull request Aug 14, 2024
stackotter pushed a commit to stackotter/helix that referenced this pull request Aug 28, 2024
kyruzic pushed a commit to kyruzic/helix that referenced this pull request Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-language-support Area: Support for programming/text languages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants