Feature request: Better C# syntax colorizer

[aeschli]

From @nfantone on August 17, 2016 14:57

On latest Insiders (1.5.0-55168276e4c0b6ec2404ac20af04eeed5f5b1000), C# syntax highlighting look very bland.

As you can see, almost 70% of the text is just plain white, with only a few keyword being blue. Tried different themes as well, and all of them colorize the same bits leaving what was white-ish in the default theme intact.

Am I doing something wrong? Is there an extension of sorts for enhancing this? I am new to C# in general, so perhaps this is to be expected? Or are there plans to support better syntax highlighting in the future?

Copied from original issue: microsoft/vscode#10619

[johanneskopf]

Same here with the latest 1.5.0 insider.
"Vector3", "Input" and "Screen" for example in my picture should definitely be colorized.

[DustinCampbell]

@aeschli: Note that this is a request for property semantic type colorization, which I do not believe is possible with VS Code today, since it relies on a syntactic grammar. Please let me know if I'm incorrect about that.

[aeschli]

@DustinCampbell You are correct, we only have syntax highlighting through text mate grammars. Some grammars go quite far as to also parse statements and expressions, but depending on the complexity of the language this is not feasible.

[cwe1ss]

@nfantone @aeschli seems like this specific screenshot is from using the "Dark" theme instead of "Dark+" (which has nicer highlighting), isn't it?

Themes can be changed in the command palette by typing "Preferences: Color Theme"

[nfantone]

@cwe1ss While it's still has plenty of room for improvement, since 1.4.0 release, colorizer greatly improved. That screenshot was taken almost a month ago, now. Not entirely sure, but it should have been taken using 1.3.0.

However, I do remember that switching themes at that moment did absolutely nothing but change the colors of what was already colorized in the shown theme.

Currently, it looks something like this (using Dark+):

It'd be great if classes whose static members are being accessed would be somewhat emphasized and not left completely white (like that Random.Range call). As an example, take a look at what Visual Studio does.

[ivanz]

Found this feature request in VSCode for exposing a way to plug-in programmatic (semantic) highlighting: microsoft/vscode#585

Until that's implemented we can only do syntax highlighting based on regular expressions, which can do only so much.

[DustinCampbell]

Thanks @ivanz. I went ahead and commented on that issue as well.

[nfantone]

@DustinCampbell One thing I can't quite get my head around is why we need semantic knowledge of an instruction to change the color of Customer in something like this: Customer.Find(). I know next to nothing about textmate, but it sure seems like something incredibly simple to implement with a regex.

The only complication I can think of is if someones uses PascalCase to name their variables or does not with classes names. But, then again, those people deserve not to have proper syntax highlighting.

If you assume some kind of coding standard, you can begin to cover a very wide range of cases without semantic shenanigans.

[DustinCampbell]

@nfantone: It's not shenanigans. Casing isn't enough to even make an educated guess. Without semantically knowing what Customer is, we have no why of knowing whether its a class, a property, a public field, etc.

Note that we actually already have all of this information with OmniSharp. We have all the same information that is used to colorize Visual Studio. We just need a way to present that information in VS Code.

[nfantone]

@DustinCampbell You are right. I know it's not "shenanigans". Bad wording, sorry. I disagree with this, tough:

Casing isn't enough to even make an educated guess.

Frankly, I believe it's more than enough. It won't be an exact match, but it'll sure be educated.

My point was (is) why aren't we colorizing those terms anyway? Why is the alternative (mostly white text) preferable? Maybe I'm alone with this opinion here, but I'd take having Customer in Customer.Find() being of a different color than Find() any day - regardless of it being a static reference, a local variable or a class attribute.

Just for reference, that is what the Javascript syntax highlighter in, say, ST3 does. See below.

Both Object and Customer are colorized different than their members.

[DustinCampbell]

We're definitely accepting pull requests. However, as we already have exactly the information to colorize, we're not willing to spend cycles adding heuristics to the reg-ex based grammar to make it look right some of the time.

JavaScript is not really a fair comparison as the standard for casing is pretty different. In C#, we'd get it wrong a lot more often.