Consider more contrast for comments

[josh-k-cb]

Description

The theme could benefit from more color contrast for comments in Typescript files, especially during diff.

Standard comment:

Diff comment:

I've personally applied these:

  "editor.tokenColorCustomizations": {
    "[Nord]": {
      "comments": "#81A1C1"
    }
  },
  "workbench.colorCustomizations": {
    "[Nord]": {
      "diffEditor.insertedTextBackground": "#5E81AC33"
    }
  }

to get this:

Standard comment:

Diff comment:

My solution might not be 'pure Nord' but I can read comments during diff, which is nice

Benefits

Readability

Possible Drawbacks

Objective opinion

[crizant]

This is exactly what i've been looking for, thanks!
Finally I can see comments in the git diff panel. I think this should be applied by default.

[svengreb]

Thank you for your patience! 🙏🏼
It‘s been a while since I had free time to focus more on Nord, and my open source projects in general, and invest time in this issue due to work-life balance.

I recently published the first “Northern Post — The state and roadmap of Nord“ announcement which includes all details about the plans and future of the Nord project, including the goal of catching up with the backlog. This issue is part of the backlog and therefore I want to triage and process it to get one step closer to a “clean state“. Read the announcement about reaching the “clean“ contribution triage state in Nord‘s discussions for more details about the goal.

Due to the scope of this issue of not being “purely technical“, or being related to the project‘s code at all, I‘ll convert it into a discussion for Nord‘s central community hub on GitHub. This way the repository is easier to maintain by separating conversations about the actual code while also making help requests, ideas and other contributions from users more visible and supportable by the community.

---

@josh-k-cb Thanks again for your contributions 🚀
Changes like this can be, like you already did, made by customizing the theme to fit your needs. The comment color was discussed and adjusted across all syntax highlighting related Nord ports so all details about the decision of the color can be read there.
I‘ll close this discussion since this is a user-side related solution.