-
Notifications
You must be signed in to change notification settings - Fork 8
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
Enhancements to snippets and syntaxes #32
Comments
@tgingold I am going through the current syntaxes to change the highlight colors for all Basically, it uses a |
@tgingold can I get permissions? I made a branch and I'm starting to commit my changes/additions to the snippets. It won't let me with the following error:
|
@GlenNicholls, the regular procedure is to fork this repo to say |
@1138-4eb Can you check out my commit for the snippets in 4ce89e4? I am adding as much as I can and would like to organize the Also, what do you think about the above comment I made about the syntax highlighting? |
@1138-4eb @tgingold I am also thinking about adding a snippet for a VHDL file header:
Obviously, the format could change, but I use that all the time. Is there value in adding this to the ghdl-ls vscode plugin, or is that too specific? |
Many people want to have an header, but rarely the same. How easily could it be customized by users ?
|
I have to investigate. I think the existing identifiers are already highlighted on my setup.
|
Hmmm, I don't think it would be an easy task. I'm not even sure that it would work with the provided snippet variables. I opened a VSC issue about allowing embedded snippets which would help. At this time, though, I don't think it would be possible for a user to customize the header without a macro or something.
Okay let me know. I have tried all default themes and turned off all VHDL extensions except ghdl-ls and they aren't highlighted at all for me. |
… takes longer than just writing it manually
Do you have an |
* Adding snippets per /issues/32 * more snippets added for /issues/32 * fixed bugs in snippets and added todos * moved todos for #32 and removed attributes since typing that word takes longer than just writing it manually * organizing a bit and moving snippets to multiple files * moved snippets to seperate files based on logical ordering and updated package.json * re-organized before merging with master for #32 * fixed file name issue and duplicate snippet in interface
The snippets and syntaxes are in rough form right now. The following should be added:
Syntaxes
The following keywords are not currently in the syntaxes
' '
'
being valid. Possibly check to make sure there's something between two single quotes. Making this a different color from the double quote would be helpful as wellSnippets
The following need to be added to snippets
entitycomponentarchitecturepackageieee librariesprocedureThe text was updated successfully, but these errors were encountered: