Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 531869 - Use more appropriate colour for INFORMATION_BACKGROUND
Summary: Use more appropriate colour for INFORMATION_BACKGROUND
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.8   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: refactorPopupBlackBg
  Show dependency tree
 
Reported: 2018-03-01 11:56 EST by Roland Grunberg CLA
Modified: 2018-03-02 13:08 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Roland Grunberg CLA 2018-03-01 11:56:05 EST
This was brought about by Bug 515755.

Basically, INFORMATION_BACKGROUND improves the background of the popup mentioned above on the dark theme but it still doesn't look ideal on Linux, Mac, or win32 with the dark theme :

Before INFORMATION_BACKGROUND :

https://bugs.eclipse.org/bugs/attachment.cgi?id=268091
https://bugs.eclipse.org/bugs/attachment.cgi?id=272947

With INFORMATION_BACKGROUND :

https://bugs.eclipse.org/bugs/attachment.cgi?id=272948
https://bugs.eclipse.org/bugs/attachment.cgi?id=272642


The following change :

In bundles/org.eclipse.ui.themes/css/dark/e4-dark_preferencestyle.css

-                 'org.eclipse.ui.workbench.INFORMATION_BACKGROUND=52,57,61'
+               'org.eclipse.ui.workbench.INFORMATION_BACKGROUND=81,86,88'

makes it look like https://bugs.eclipse.org/bugs/attachment.cgi?id=272949 .

The main thing to investigate is whether this has any negative effect on all the other places that use INFORMATION_BACKGROUND. The basic assumption is that most things styled with INFORMATION_BACKGROUND should be some kind of shell or, something that may be placed on a shell. In either case they should probably have the same background colour as given to shells on the dark theme, which is defined in bundles/org.eclipse.ui.themes/css/dark/e4-dark_globalstyle.css as #515658 (rgb 81,86,88).

I wouldn't say this blocks Bug 515755 since using INFORMATION_BACKGROUND is still an improvement regardless, but it would certainly look better if this change was considered.
Comment 1 Lars Vogel CLA 2018-03-01 12:05:50 EST
Can you provide a Gerrit so that we can test it in a runtime IDE?
Comment 2 Eclipse Genie CLA 2018-03-01 12:27:21 EST
New Gerrit change created: https://git.eclipse.org/r/118445