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

Update grammar and highlight queries for slint grammar rewrite #5290

Closed
wants to merge 1 commit into from

Conversation

jrmoulton
Copy link
Contributor

This is just updating the grammar rev and queries to be updated to the latest slint grammar.

@jrmoulton
Copy link
Contributor Author

Ahh. I accidentally have an extra unrelated commit in here. I'll fix that

Update slint highlight queries for grammar rewrite

Revert "Add scrolling down past half the buffer height"

This reverts commit 4155026.
@jrmoulton
Copy link
Contributor Author

Fixed

Copy link
Member

@the-mikedavis the-mikedavis left a comment

Choose a reason for hiding this comment

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

It looks like local.scm will need to be updated as well since it contains a node that doesn't exist in the new version of the grammar: https://github.com/helix-editor/helix/actions/runs/3777270745/jobs/6422763958

Comment on lines 153 to 154
(ternary_expression [":" "?"] @keyword.control.conditional) No newline at end of file
(ternary_expression [":" "?"] @conditional)
Copy link
Member

Choose a reason for hiding this comment

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

Helix uses different captures for highlighting than neovim - https://docs.helix-editor.com/master/themes.html#syntax-highlighting- so this change should be undone

There are some other non-existent scopes that should be changed too, for example @float should become @constant.numeric.float

@kirawi kirawi added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. A-language-support Area: Support for programming/text languages labels Dec 28, 2022
@lauglam
Copy link

lauglam commented Jun 26, 2023

Has there been any progress on this issue?

@jrmoulton
Copy link
Contributor Author

I haven’t worked on this in a while. There isn’t much that needs changed at this point but the queries need to be updated for helix

@erasin erasin mentioned this pull request Aug 10, 2023
@gabydd
Copy link
Member

gabydd commented Aug 13, 2023

Closed by #7893

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 S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants