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

Missing parts of the syntax highlighting #1

Open
MegaIng opened this issue Jun 20, 2019 · 14 comments
Open

Missing parts of the syntax highlighting #1

MegaIng opened this issue Jun 20, 2019 · 14 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@MegaIng
Copy link
Member

MegaIng commented Jun 20, 2019

Use this issue to mention all parts where syntax-highlighting is wrong/not as wished/missing in the options menu.

@MegaIng MegaIng added enhancement New feature or request help wanted Extra attention is needed labels Jun 20, 2019
@MegaIng MegaIng self-assigned this Jun 20, 2019
@denizetkar
Copy link

denizetkar commented Nov 5, 2019

Hi, thanks for providing us 'lark' users with jetbrains syntax highlighting! I would like to make a few enhancement suggessions:

  • Use special highlight colors for non-terminal and terminal symbols.
    lark2

  • Attach *.lark files with a distinctive icon (just like how python files are attached with a python icon).
    lark

Those 2 would be the first improvements that I would love to see. 🙂

@MegaIng
Copy link
Member Author

MegaIng commented Nov 5, 2019

Do you have an idea what kind of symbol could be used? Adding it is not so much work, I am just bad at designing (it would have to be an svg file if you would like to create it as outlined here)

Also, what kind of default colors would you propose for terminal/non-terminal?

@MegaIng
Copy link
Member Author

MegaIng commented Nov 5, 2019

I just realized that you can use different highlighting for tokens and rules via Settings->Editor->Color Scheme->Lark.

@denizetkar
Copy link

denizetkar commented Nov 6, 2019

I just realized that you can use different highlighting for tokens and rules via Settings->Editor->Color Scheme->Lark.

Thank you for this useful information! I think it would be a good idea to mention about such features in the main page.

Do you have an idea what kind of symbol could be used? Adding it is not so much work, I am just bad at designing (it would have to be an svg file if you would like to create it as outlined here)

Also, what kind of default colors would you propose for terminal/non-terminal?

Something like the following could look nice, although it still has a lot of room for improvement:
Lark.zip

(NOTE: i used this website for svg icon design)

@MegaIng
Copy link
Member Author

MegaIng commented Nov 6, 2019

After looking around a bit, I found a good candidate. @erezsh Do you have the logo/banner of this organization in some other form than a png? (best case .svg file) and would you think this is a good idea (using that as the icon of the plugin, and as the symbol for the files (as file-type))?

@erezsh
Copy link
Member

erezsh commented Nov 7, 2019

The bird on its own could make a nice icon. What dimensions are we talking?

@MegaIng
Copy link
Member Author

MegaIng commented Nov 7, 2019

Here is the information for a plugin icon. Basically: a .svg file, 40x40 px (but it since it is svg it will be scaled up).
There should be two versions, one for light backgrounds, one for dark backgrounds)

Here is the information for the file-type icon. Best case again .svg, can also be a .png. 16x16px, again a light/dark version, for each '@2x' scaled versions should exist.

@naretto
Copy link

naretto commented Jan 31, 2020

First of all, thank you for this. Amazing effort.

My lark file has this line:

%import common (LETTER, DIGIT, INT, WS, ESCAPED_STRING, FLOAT)

The line gets many errors:

  • each comma inside the parenthesis
  • the space between "common" and "("
  • the closing ")"

I'm new to lark, so maybe I could improve the error line, but this has been working on the project long before I joined.

Thank you again.

@erezsh
Copy link
Member

erezsh commented Jan 31, 2020

Yes, that's probably because it's a relatively new syntax.

If anyone decides to add it, I suggest to support the extended syntax:

%import common (a -> b, C -> D, and -> etc)

@MegaIng
Copy link
Member Author

MegaIng commented Feb 1, 2020

@naretto I will try to fix this soon (maybe today, maybe in the next week)

@erezsh Could you try to keep the lark.lark file up-to-date and inform me about any changes via a mention? (Or is there a way to watch an individual file?)

(And what is about the banner icon we talk about in the past?)

@erezsh
Copy link
Member

erezsh commented Feb 1, 2020

@MegaIng Sure, I can do that. Maybe I can just notify everyone in the highlighters group whenever there's a syntax change. I'll get to the icon at some point.

@MegaIng
Copy link
Member Author

MegaIng commented Feb 1, 2020

@erezsh Thanks!

@MegaIng
Copy link
Member Author

MegaIng commented Feb 1, 2020

@naretto In the latest commit it should be working. Tell me if you find any bugs. (For acutal bugs, please add a new issue)

@ornariece
Copy link

just saw this issue, after creating another one. sorry about that.
here it is: #4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants