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

Support C/CPP syntax highlighting by default #4816

Closed
sidazhang opened this issue Apr 5, 2019 · 7 comments
Closed

Support C/CPP syntax highlighting by default #4816

sidazhang opened this issue Apr 5, 2019 · 7 comments
Labels
cpp issues related to the C/C++ language enhancement issues that are enhancements to current functionality - nice to haves help wanted issues meant to be picked up, require help textmate issues related to the textmate grammars

Comments

@sidazhang
Copy link

Description

Currently theia does not support CPP syntax highlighting out of the box.

Is it possible to support CPP syntax highlighting. I am not referring to CPP language server. A simple CPP file syntax highlight is sufficient out of the box

@sidazhang sidazhang changed the title Support CPP syntax highlighting by default Support C/CPP syntax highlighting by default Apr 5, 2019
@akosyakov akosyakov added cpp issues related to the C/C++ language enhancement issues that are enhancements to current functionality - nice to haves help wanted issues meant to be picked up, require help textmate issues related to the textmate grammars labels Apr 5, 2019
@akosyakov
Copy link
Member

We can move https://github.com/theia-ide/theia/blob/master/packages/cpp/src/browser/cpp-grammar-contribution.ts to textmage-grammar extension. PRs are welcomed.

@simark @marechal-p @thegecko hope it is fine with you

@thegecko
Copy link
Member

thegecko commented Apr 5, 2019

No concerns.

@arekzaluski ?

@vince-fugnitto
Copy link
Member

@akosyakov one thing, I'm wondering if it might be wise that the @theia/languages extension explicitly depend on the @theia/textmate-grammars extension.

This would mean that any language extension such as @theia/cpp would not need to depend on @theia/textmate-grammars itself and can benefit from the textmate support. At the moment only the browser and electron example applications currently import the textmate extension. If a user depends on the cpp extension, then he will be sure to have the textmate support as well.

@akosyakov
Copy link
Member

akosyakov commented Apr 5, 2019

We want to get rid of @theia/textmate-grammars eventually, and use vscode built-in extensions for textmate grammars. I don't want to couple it with other extensions because of it.

@vince-fugnitto
Copy link
Member

Closed by #5803

@svkr2k
Copy link

svkr2k commented Apr 23, 2020

Hello all,
Kindly confirm, is @theia/textmate-grammars enough for C/C++ syntax highlight and @theia/cpp is not needed anymore ?

@vince-fugnitto
Copy link
Member

Kindly confirm, is @theia/textmate-grammars enough for C/C++ syntax highlight and @theia/cpp is not needed anymore ?

@svkr2k the extension @theia/textmate-grammars has been deprecated, syntax-hihglighting for C/C++ is contributed by the vscode builtin extension:

"vscode-builtin-cpp": "http://open-vsx.org/api/vscode/cpp/1.39.1/file/vscode.cpp-1.39.1.vsix",

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp issues related to the C/C++ language enhancement issues that are enhancements to current functionality - nice to haves help wanted issues meant to be picked up, require help textmate issues related to the textmate grammars
Projects
None yet
Development

No branches or pull requests

5 participants