-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Add Djot support #12562
base: master
Are you sure you want to change the base?
Add Djot support #12562
Conversation
If it would make the reviewing process easier for someone who is unfamiliar with djot, here is an example file. I've had to change the extension to upload it to GitHub so it will need to be changed back to Additionally, here are some screenshots demonstrating the highlights: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad to see this, as I enjoy using Djot as well!
I was waiting for treeman/tree-sitter-djot#55 to be merged before starting this PR, but this already looks great to me!
b723e2c
to
d8a87af
Compare
The update to the tree-sitter-djot grammar has been merged in treeman/tree-sitter-djot#56, which includes some breaking changes. It won't be separated into two separate grammars, but the highlights will need to be adjusted regardless. I'll begin working on the necessary changes today when I get the time. |
@zetashift sorry to call you back to this so soon! The highlights and grammar have been updated to the new version of tree-sitter-djot. Could we maybe get this PR marked with the #9458 has been merged, so I'll also need to swap out the queries to be inverse. I've already maintained that change separately so I'll just quickly make that adjustment later today. |
This change adds support for the Djot markup language. The grammar used is tree-sitter-djot and the highlights & injections have been adapted for Helix's scopes.
There are 4 highlight queries with TEMP comments above them as they are semantically significant but I don't feel they have appropriate existing scopes.
In preparation for #9458 I will be maintaining a version with inverse precedence.
Closes #12264
Closes treeman/tree-sitter-djot#10