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

Low contrast between background colour and excluded folders #57

Open
j1ah0ng opened this issue Jun 2, 2019 · 7 comments
Open

Low contrast between background colour and excluded folders #57

j1ah0ng opened this issue Jun 2, 2019 · 7 comments

Comments

@j1ah0ng
Copy link

j1ah0ng commented Jun 2, 2019

Deselected excluded folders have really low contrast that makes their names difficult to read:

image

Once highlighted, their appearance improves:

image

@arcticicestudio
Copy link
Contributor

Hi @sushisharkjl 👋, thanks for your contribution 👍
I guess this is a problem based on two factors: The foreground color for excluded files and folder uses a low contrast by design. This decision has been made so the theme assists you to keep the focus on the important things: the source code as well as the relevant files/folders when seen for the tree view. The excluded data is excluded with a reason: you shouldn't care about it 😄

Anyway, the second fact is that it looks like you're using IntelliJ's File Colors features that highlights some files in various UI elements with different background colors. You can take a look at #53 that changed the style for this features (release in version 0.6.0) and also follow the instructions in the linked JetBrains documentation to disable the features if you like. I've never used it and in my opinion the UI feel way more cleaner + the files/folders can be read better again.

Let me know if it helps to make it useable again for you 👍

@Sushisource
Copy link

I'll chip in here and say this struck me as a bit painful too. I understand the philosophy, but given that File Colors is enabled by default, probably makes sense to make these a little bit more legible. De-emphasizing makes perfect sense, but not to the point of nearly-impossible-to-read. Personally, I find the highlights quite useful for test folders.

The fix in #53 doesn't seem to be totally working, as I am using 0.6.0 and get the exact look as in OP's screenshot. I would expect the text on the excluded folders to be white as in the fix, but it's still greyish for me.

Additionally, the little expansion arrows are really really hard to see on top of that blue background when items are highlighted.

Love your theme! Thanks for your hard work!

@arcticicestudio
Copy link
Contributor

arcticicestudio commented Aug 1, 2019

@Sushisource I totally agree, but unfortunately there are no theme keys to explicitly set the foreground color of ignored/excluded files in a tree view when a file color has been applied. The only solution would be to change the general ignore/exclude foreground color which again would make it impossible to differ between a normal and excluded folder when the user has disabled the File Colors feature. Also even when File Colors are enabled by default, the background color is not inherited from the active UI theme but the default and „hardcoded“ values of the parent theme Darcula.

In short: As long as there is no way to apply specific colors/styles when a specific condition is met this is a edge case that a theme author can not handle.

There are other edge case that are hard to handle since the user can choose any background color.
How to determine if the chosen background color is dark or bright? Using a bright color like white in combination with e.g. nord4 as foreground color would make text almost invisible. On the other hand in situations like now where the default/fallback Darcula values are used the dark foreground color nord3 makes it also hard to read.

This pseudo code shows an possible solution that is necessary in order to fix the problem that must be added by JetBrains to the theme API in any way exposed to the JSON theme interface:

var ignore_or_excluded_foreground_color = "nord3"
var matching_file_color_scope_background_color = getMatchingFileColorBgColor()

if file_colors {
  if calcBrightnessRatio(matching_file_color_scope_background_color) > "50%" {
    ignore_or_excluded_foreground_color = "nord3"
  } else {
     ignore_or_excluded_foreground_color = "nord4"
  }
}

I hope this makes the situation a bit more clear, it's not as easy as you think and unfortunately as a theme author I depend on a solution on the side of JetBrains 😞

@jgrossi
Copy link

jgrossi commented Aug 29, 2019

@sushisharkjl @Sushisource I have the same issue. the only solution was unchecking the "ignored" "file status color":
image

@arcticicestudio
Copy link
Contributor

@jgrossi Oh, I never saw this screen in previous IDE versions before 😲
I'll definitely need to check this. The theme scheme itself doesn't expose any keys to customize the colors so I'll initialize and commit all IDE setting into a Git repository and test which keys are added when using this UI view 😄

@adampatterson
Copy link

Disabling "File Colors" solved it for me, I was able to leave the Version status colors as is.

image

@r4j4h
Copy link

r4j4h commented Jun 8, 2021

not sure where this should go but it seems related to this, so I wanted to emphasize this proposed solution on the official plugins reviews: https://imgur.com/r7G5WgX from https://plugins.jetbrains.com/plugin/10321-nord/reviews#review=53562-53563

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

No branches or pull requests

6 participants