TS and JS inconsistencies

[svipas]

Interpolated string literals:

It's not working in TS because it doesn't contain source.ts and source.tsx here: https://github.com/arcticicestudio/nord-visual-studio-code/pull/114/files

JSDoc:

TS

JS

In my opinion it should be as it's in TS and type annotations like string should be colored.

[arcticicestudio]

Thanks for your contribution 👍

This is a known problem and a note has also been added to the backlog to fix the missing syntax scopes.
I'll take this issue as reference instead of converting the note to an issue 😄

[svipas]

Also since it's related I found another issue with class initialization.

As you can see in JS it's not colorized, but in TS it is.

[svipas]

@arcticicestudio I know you're busy and I can help you. If needed I can solve these issues and submit a PR, I can even try to make screenshots before -> after.

[arcticicestudio]

@svipas Unfortunately my free time is indeed very limited and I try to get the transition of all Nord port projects to the new website done when there is some time so I really appreciate every help and contribution even more than usual 😄
Feel free to go ahead and submit a PR, I'll try to find some time on a weekend to review it. Also screenshots are a nice-to-have, but please do not do any extra work to create them, I always create missing screenshots anyway as soon as I write changelogs and prepare the release version deployments.

I guess most existing selectors for JavaScript (source.js) can be extended by adding the source.ts scope, but as far as I've used TypeScript there might also be some new selectors, e.g. to highlight interfaces or the primary or abstract types and any other element that is specific to TypeScript.

For React JSX the selectors should work fine when the .jsx and .tsx scopes are appended for selectors that are relevant and might not be highlighted correctly yet.

[svipas]

@arcticicestudio Created a PR: #142

[svipas]

Also, found a bug with meta.brace.round this scope was missing, and if you take a look at colors of () function calls, you will see they are #d8dee9 instead of #eceff4. This is also fixed in the PR mentioned above.

[svipas]

Found issue with meta.brace.square too similar as mentioned above and pushed a new commit to a PR. I guess right now everything is OK and there will be no problems :)

[razvanbadileanu]

That's indeed a nice theme and I want to congratulate you for this, but it seems there is a small issue and I'm referring to the same topic. A small one but annoying and it's related again to typescript, in template string expression color representation differs from JS version, this one been more readable.

JS version

TS version