-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Comments
Hi @sushisharkjl 👋, thanks for your contribution 👍 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 👍 |
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! |
@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. 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 😞 |
@sushisharkjl @Sushisource I have the same issue. the only solution was unchecking the "ignored" "file status color": |
@jgrossi Oh, I never saw this screen in previous IDE versions before 😲 |
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 |
Deselected excluded folders have really low contrast that makes their names difficult to read:
Once highlighted, their appearance improves:
The text was updated successfully, but these errors were encountered: