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

Poor contrast between 'added block' background and comments in comparison view #103

Closed
bluebartle opened this issue Sep 26, 2019 · 5 comments · Fixed by #159
Closed

Poor contrast between 'added block' background and comments in comparison view #103

bluebartle opened this issue Sep 26, 2019 · 5 comments · Fixed by #159

Comments

@bluebartle
Copy link
Contributor

bluebartle commented Sep 26, 2019

When using the comparison view, the contrast between the 'added block' background colour and the comment colour makes reading the comments difficult (or at least, uncomfortable). This particular background colour makes reading anything at least a little uncomfortable, and I tend to select-all in order to avoid it, but I find it most difficult with comments.

Comments:

Screenshot 2019-09-26 at 09 26 57

General:

Screenshot 2019-09-26 at 09 36 04

@arcticicestudio
Copy link
Contributor

Hi @ratskinmahoney 👋, thanks for your contribution 👍
This was a known problem some years ago and was improved in #25, but I can see that this might be a problem. I currently not sure if you're using the latest version of the theme plugin since the comment color was also improved in #42 (released with version 0.4.0), but the color on your screenshots looks not like the updated one.

  1. Please make sure you're using the latest plugin version.
  2. Please check if you're using a copy of the theme which gets automatically created by the IDE when a setting of the currently active editor or UI theme gets changed by the user. You can follow the instructions I've posted in Goland 2019.2.1 display wrong color for go code #100 (comment) which also solved the problem for the user.

I'll also test the diff view next time I push a PR. If this style is still used in the latest version, I think I'll port the improved styles implemented for the VS Code theme.

@bluebartle
Copy link
Contributor Author

bluebartle commented Sep 27, 2019

Thanks for the prompt and detailed reply, @arcticicestudio.

The version of the plugin I'm using is 0.8.1 - apologies for missing this information out in the initial report:
Screenshot 2019-09-27 at 08 13 09

I've followed the instructions in the comment you linked above:

  • Navigated to ~/Library/Preferences/<app>/colors/ (I'm on mac)
  • Removed _@user_Nord.icls
  • Started IntelliJ Idea
  • In settings switched theme away from Nord, applied, and then switched the theme back to Nord and applied
    I've also tried uninstalling and reinstalling the plug-in, but with the same effect.

The colours still look the same, I'm afraid:
Screenshot 2019-09-27 at 08 17 15

Here is what they look like normally:
Screenshot 2019-09-27 at 11 08 44

PS: I'd be happy with the VS Code theme colours suggested. Cheers!

@nmaves
Copy link

nmaves commented Jun 6, 2020

I am still seeing this issue as well with 11.2

@arcticicestudio
Copy link
Contributor

This is still a pending topic one a large list, but everyone can feel free to submit a PR using the linked PR from the VS Code port as reference. Unfortunately the proposal microsoft/vscode#71663 was closed/not accepted to improve the theme API within VS Code itself so I guess there's no other way around than adapting the adjusted styles from Nord Visual Studio Code.

@bluebartle
Copy link
Contributor Author

bluebartle commented Jun 20, 2020

Thanks for the update, @arcticicestudio.

Edit: Opened a PR and moved the comment to there: #159

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants