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 233624 - [dark] [nimbus] patch in editor is not readable due to colors
Summary: [dark] [nimbus] patch in editor is not readable due to colors
Status: VERIFIED FIXED
Alias: None
Product: utilities
Classification: Unclassified
Component: Diff (show other bugs)
Version: 7.4
Hardware: PC Windows 7
: P3 normal (vote)
Assignee: Tomas Stupka
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-07-29 22:45 UTC by markiewb
Modified: 2013-08-25 12:03 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Shows issue using Dark NImbus (46.81 KB, image/png)
2013-07-29 22:45 UTC, markiewb
Details
diff default colors in Norway Today (88.57 KB, image/png)
2013-07-30 07:44 UTC, Ondrej Vrabec
Details
Diff vs. Patch colors (137.95 KB, image/png)
2013-07-30 18:35 UTC, markiewb
Details
weird colors (22.01 KB, image/png)
2013-07-31 07:42 UTC, Ondrej Vrabec
Details
suggestion (24.63 KB, image/png)
2013-07-31 07:49 UTC, Ondrej Vrabec
Details

Note You need to log in before you can comment on or make changes to this bug.
Description markiewb 2013-07-29 22:45:53 UTC
Created attachment 137973 [details]
Shows issue using Dark NImbus

Same issue like https://netbeans.org/bugzilla/show_bug.cgi?id=231721 happens using Dark Nimbus

* open patch file

ACTUAL: blue text on dark background - see screenshot
EXPECTED: better contrast 

---
Dark NImbus with Norway
Product Version: NetBeans IDE Dev (Build 201307262300)
Java: 1.7.0_25; Java HotSpot(TM) 64-Bit Server VM 23.25-b01
Runtime: Java(TM) SE Runtime Environment 1.7.0_25-b16
System: Windows 7 version 6.1 running on amd64; Cp1252; en_US (nb)
Comment 1 Ondrej Vrabec 2013-07-30 07:44:18 UTC
what editor color scheme do you use? Diff editor colors are not affected by selected laf but rather by the editor colors profile. In Norway Today dark theme it's ok. See the screenshot.
Please attach the same screenshot from your IDE.
Comment 2 Ondrej Vrabec 2013-07-30 07:44:21 UTC
Created attachment 137982 [details]
diff default colors in Norway Today
Comment 3 Ondrej Vrabec 2013-07-30 07:54:00 UTC
maybe the colors in Norway scheme are cached, please try to click on Restore or run with fresh userdir and report. Thanks
Comment 4 markiewb 2013-07-30 18:35:45 UTC
Created attachment 138015 [details]
Diff vs. Patch colors

(In reply to comment #3)
> maybe the colors in Norway scheme are cached, please try to click on Restore or
> run with fresh userdir and report. Thanks

I tried it and in your screenshot there is the same issue. I am complaining about the "blue color on dark background". This color combination isn't readable IMHO.

I propose to use the same colors as used in diff view. See screenshot. Does this make sense?
Comment 5 markiewb 2013-07-30 18:37:27 UTC
(In reply to comment #4)
> I propose to use the same colors as used in diff view. See screenshot. 

Some more detailed: I propose to use the same colors (foreground AND background) as used in diff view.
Comment 6 Ondrej Vrabec 2013-07-31 07:42:18 UTC
> (In reply to comment #4)
> Some more detailed: I propose to use the same colors (foreground AND background) as used in diff view.
It doesn't seem right, it is readable, yes. But for me the removed/added sections almost blend together and i cannot tell the difference between them. I suggest changing blue to green (similar to this http://www.malloc.co/wp-content/uploads/2012/10/Screen-Shot-2012-10-17-at-10.12.56-PM.png)
Comment 7 Ondrej Vrabec 2013-07-31 07:42:21 UTC
Created attachment 138036 [details]
weird colors
Comment 8 Ondrej Vrabec 2013-07-31 07:49:44 UTC
what about diffeditor2.png as a suggestion? Changing blue to green?
Comment 9 Ondrej Vrabec 2013-07-31 07:49:46 UTC
Created attachment 138037 [details]
suggestion
Comment 10 markiewb 2013-07-31 11:05:12 UTC
(In reply to comment #8)
> what about diffeditor2.png as a suggestion? Changing blue to green?
Yes, the contrast is much better! Do you like to change the diff-colors too, so that it looks consistent?!

> http://www.malloc.co/wp-content/uploads/2012/10/Screen-Shot-2012-10-17-at-10.12.56-PM.png
The contrast in the screenshot is also OK.
Comment 11 Ondrej Vrabec 2013-07-31 11:15:52 UTC
(In reply to comment #10)
> Do you like to change the diff-colors too, so
> that it looks consistent?!
What color do you mean? Background colors in the diff view on https://netbeans.org/bugzilla/attachment.cgi?id=138015 ? They are also green/red although much darker.
Comment 12 Ondrej Vrabec 2013-07-31 14:53:44 UTC
fix: http://hg.netbeans.org/core-main/rev/66585820104a
Comment 13 Quality Engineering 2013-08-02 02:34:27 UTC
Integrated into 'main-silver', will be available in build *201308012300* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/66585820104a
User: Ondrej Vrabec <ovrabec@netbeans.org>
Log: #233624 - [dark] [nimbus] patch in editor is not readable due to colors
Comment 14 markiewb 2013-08-25 12:03:58 UTC
Verified in 201308240953