This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 258789 - [dark] modified symbols in diff are invisible
Summary: [dark] modified symbols in diff are invisible
Status: RESOLVED FIXED
Alias: None
Product: utilities
Classification: Unclassified
Component: Diff (show other bugs)
Version: 8.2
Hardware: All All
: P3 normal (vote)
Assignee: Ondrej Vrabec
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-12 11:52 UTC by Vladimir Voskresensky
Modified: 2016-04-16 01:50 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
impossible to distinguish ImEE from IjEE (21.86 KB, image/png)
2016-04-12 11:54 UTC, Vladimir Voskresensky
Details
Before (38.24 KB, image/png)
2016-04-13 12:47 UTC, ilia
Details
After (38.26 KB, image/png)
2016-04-13 12:47 UTC, ilia
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Vladimir Voskresensky 2016-04-12 11:52:24 UTC
When I switch to dark L&F (or Norway Today scheme) then it is impossible to see in "Diff" view changed symbols inside the line, because of used color with very dark bg.
Please, change them to make colors visible in Norway Today scheme

Thanks,
Vladimir.
Comment 1 Vladimir Voskresensky 2016-04-12 11:54:29 UTC
Created attachment 159230 [details]
impossible to distinguish ImEE from IjEE
Comment 2 Ondrej Vrabec 2016-04-12 12:16:11 UTC
Tools -> Options -> Fonts & Colors -> Diff tab -> Color of Changed Text. Set it to anything you want. It should help.
Comment 3 Vladimir Voskresensky 2016-04-12 12:21:01 UTC
Thanks. I know where to change. 
This is just an issue to make default color better like https://netbeans.org/bugzilla/show_bug.cgi?id=258606
You can use nice resource http://paletton.com/ to find good composition of colors, because you need to find color which is good for dark theme + selected/not_selected current line color should be considered as well
Comment 4 Ondrej Vrabec 2016-04-12 13:05:43 UTC
Sure, if you find a good and proper colour I am more than happy to change the default. But I am sorry I do not have time to inspect all possible variations of the background/foreground colours. Or if you know a UX person who could help us, please add him/her on CC.
Comment 5 Vladimir Voskresensky 2016-04-12 16:21:31 UTC
Ilia, could you, please, suggest good colors for Dark Theme in Diff View.
Thanks!
Comment 6 ilia 2016-04-13 12:47:37 UTC
Created attachment 159249 [details]
Before
Comment 7 ilia 2016-04-13 12:47:55 UTC
Created attachment 159250 [details]
After
Comment 8 ilia 2016-04-13 12:53:35 UTC
I suggest #242F65 as a new color.
It's a bit lighter than colors of the removed and added text, but it's only way to make it distinguishable from BG color and Highlighted caret row color.

Also the color of the removed text worth to be changed to #381E1E


So, 
Changed text: #202833 -> #242F65
Removed text: #332024 -> #381E1E

Also note that "Color of Unapplied Text When Merging" and "Color if Unresolved Conflict" should be changed to corresponding colors too.

Vladimir, let me know what you and users think about these colors.
Comment 9 Ondrej Vrabec 2016-04-13 13:56:20 UTC
Are you sure that blue background with light-blue text is contrast enough? What about poor-sighted developers? Is it OK?
Comment 10 Ondrej Vrabec 2016-04-14 18:34:11 UTC
fix: http://hg.netbeans.org/core-main/rev/b5fa3e7d79ff
Comment 11 ilia 2016-04-15 08:24:21 UTC
Thanks, Ondrej!

It seems that the color scheme is not suitable for use by poor-sighted developers. But I'm not sure it should be suitable, because there are a lot of types of сolor blindness and one color scheme can't be right for all.

So I took the most applicable color to the existing color scheme, considering that the "changed" is usually associated with a blue color. If we could change color to e.g. purple, things will get better.
Comment 12 Quality Engineering 2016-04-16 01:50:12 UTC
Integrated into 'main-silver', will be available in build *201604160001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/b5fa3e7d79ff
User: Ondrej Vrabec <ovrabec@netbeans.org>
Log: #258789 - [dark] modified symbols in diff are invisible
Change diff colors according to the cnd request