Bug 551532 - [CSS] Fix ExpandableComposite, Section in dark theme
Summary: [CSS] Fix ExpandableComposite, Section in dark theme
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.13   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Benedikt Kuntz CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
: 531532 (view as bug list)
Depends on: 549916
Blocks: 551494
  Show dependency tree
 
Reported: 2019-09-26 13:49 EDT by Lars Vogel CLA
Modified: 2019-11-27 07:58 EST (History)
5 users (show)

See Also:


Attachments
POM Editor in Dark Mode (4.21 KB, image/png)
2019-09-27 09:20 EDT, Benedikt Kuntz CLA
no flags Details
MANIFEST.MF Editor dark theme (3.88 KB, image/png)
2019-10-01 10:27 EDT, Benedikt Kuntz CLA
no flags Details
Screenshot (88.54 KB, image/png)
2019-10-01 12:18 EDT, Lars Vogel CLA
no flags Details
Hower issue (1.96 MB, video/quicktime)
2019-10-02 10:52 EDT, Matthias Becker CLA
no flags Details
Manifest Dark on Win/Linux (53.07 KB, image/png)
2019-10-05 06:09 EDT, Benedikt Kuntz CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2019-09-26 13:49:24 EDT Comment hidden (obsolete)
Comment 1 Lars Vogel CLA 2019-09-26 13:52:29 EDT
Once Bug 549916 is fixed, we should fix our light and dark theme issues in this area.
Comment 2 Eclipse Genie CLA 2019-09-27 09:18:54 EDT
New Gerrit change created: https://git.eclipse.org/r/150248
Comment 3 Benedikt Kuntz CLA 2019-09-27 09:20:57 EDT
Created attachment 280071 [details]
POM Editor in Dark Mode

For dark mode, i use the new stylings now (see attachment).
However, which issues are there in the light theme?
Comment 4 Benedikt Kuntz CLA 2019-09-27 09:30:19 EDT
I think this bug is related to
https://bugs.eclipse.org/bugs/show_bug.cgi?id=531532
Comment 5 Matthias Becker CLA 2019-09-27 10:29:14 EDT
(In reply to Eclipse Genie from comment #2)
> New Gerrit change created: https://git.eclipse.org/r/150248

I just tested this with the plugin.xml editor. On the dependencies tab you still see
that the colours of the "Automated Management of Dependencies" are not correct.
Just hover over and expand and collapse.
Comment 6 Benedikt Kuntz CLA 2019-10-01 06:48:14 EDT
I guess you mean that the hover color is too dark?
In the new patch set I used a lighter color. But I do not use the dark theme myself neither I am familiar with the colors used in the themes so if you have any suggestions just feel free to mention them!
Comment 7 Benedikt Kuntz CLA 2019-10-01 10:27:13 EDT
Created attachment 280121 [details]
MANIFEST.MF Editor dark theme

See screenshot above (Automated Management... label is with mouse hovering)
Comment 8 Lars Vogel CLA 2019-10-01 12:18:02 EDT
Created attachment 280124 [details]
Screenshot

Looks already way better. 

I suggest we use the same color in both places, the new one looks better to me that the blue one. I think this is the #cccccc; color code Can you update the change?
Comment 10 Lars Vogel CLA 2019-10-02 10:43:05 EDT
Matthias, please reopen if your hover issue still exists, could not reproduce.

Benedikt, thanks!
Comment 11 Matthias Becker CLA 2019-10-02 10:52:16 EDT
Created attachment 280138 [details]
Hower issue

Problem with hover still persists
Comment 12 Matthias Becker CLA 2019-10-02 10:52:48 EDT
It's still the same
Comment 13 Benedikt Kuntz CLA 2019-10-05 06:09:36 EDT
Created attachment 280169 [details]
Manifest Dark on Win/Linux

(In reply to Matthias Becker from comment #12)
> It's still the same

You are using Linux, I assume? I cannot reproduce this on my machine, but I was wondering why the labels are slightly blue in your screen capture, on my (windows) machine they are just grey/white. Using the dark theme [linux] I get the same colors as in Lars' screenshot.
Attached screenshots (left: windows dark theme, right: linux dark theme on windows)
Don't know if this is intended but seems somewhat strange in my eyes...
Comment 14 Dani Megert CLA 2019-10-10 09:37:36 EDT
Removed milestone as no one reacted to https://www.eclipse.org/lists/eclipse-dev/msg11217.html.
Comment 15 Benedikt Kuntz CLA 2019-11-07 02:42:42 EST
*** Bug 531532 has been marked as a duplicate of this bug. ***
Comment 16 Lars Vogel CLA 2019-11-08 04:32:40 EST
I suggest to move the hover issue to a new bug and close this one.

Please add to N&N.
Comment 17 Benedikt Kuntz CLA 2019-11-08 05:01:30 EST
Opened bug 552829 for the hovering issue.

I mentioned it already in N&N for bug 549916, see https://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/?id=fa40a43d480393c09f79a7654f4c44d77f18ecd4
Think that should be sufficient, what do you think, Lars?
Comment 18 Lars Vogel CLA 2019-11-08 05:02:05 EST
(In reply to Benedikt Kuntz from comment #17)
> Opened bug 552829 for the hovering issue.
> 
> I mentioned it already in N&N for bug 549916, see
> https://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/
> ?id=fa40a43d480393c09f79a7654f4c44d77f18ecd4
> Think that should be sufficient, what do you think, Lars?

+1, sorry for asking for it again.