eclipse-platform / eclipse.platform.ui

Eclipse Platform
https://projects.eclipse.org/projects/eclipse.platform
Eclipse Public License 2.0
81 stars 189 forks source link

modern light theme: secondary views with wrong contrast #2200

Open jukzi opened 3 months ago

jukzi commented 3 months ago

As far as i understand every View but the Editor is supposed to have low contrast but currently it is a wild mixture of shades

Examples:

JUnit Stack trace part: image History VIew: image Problem Views, Error Log Title: image image Console: image

Also the items in the Problems View have even are darker background then the other views image

merks commented 3 months ago

For EMF-based editors, the properties view is a principle focal point when editing a model. Here is how things used to look:

image

This is how it looks now:

image

It's hard to tell what really has focus because there's always a blue under/over-line. The underlining of the tabbed page in the eidtor is different from the rest of the underlining, and changes color depending on focus. The breadcrumbs have some white stuff in them that I never seen before. Where did that come from?

Ranting constructively, this is one of the things that drive me to despair with theming. I might set a color, but something else will change that color and in the end I have no clue of how I can set colors such that all these different themes won't produce something completely unreadable in some cases. Basically one has to test every editor, view, and dialog on every theme, and even then I'm often left wondering, how do I fix this?

BeckerWdf commented 3 months ago

The breadcrumbs have some white stuff in them that I never seen before. Where did that come from?

@mvm-sap: Can you answer this question?

BeckerWdf commented 3 months ago

Ranting constructively

I like that term ;-)

this is one of the things that drive me to despair with theming. I might set a color, but something else will change that color and in the end I have no clue of how I can set colors such that all these different themes won't produce something completely unreadable in some cases. Basically one has to test every editor, view, and dialog on every theme, and even then I'm often left wondering, how do I fix this?

I really agree.

akurtakov commented 3 months ago

That's the biggest issue with theming - one either has to set EVERY color everywhere or such discrepancies will happen. It's one of the reasons why SWT by default tries to not hardcode colors but rather use the respective OS defined colors as we would most likely never have the manpower and designing capabilities of an OS developing company.

mvm-sap commented 3 months ago

The breadcrumbs have some white stuff in them that I never seen before. Where did that come from?

Yes, we wanted the background of breadcrumbs to have the same color as the text editor, hence set the background of composites in the compatibility editor to white. But, each item in the breadcrumb is not picking that as jdt has set to grey in their theming extension. I can revert those changes, so that it is completely grey and each plugin has to customize it with white background for breadcrumbs in their respective CSS extension

BeckerWdf commented 1 month ago

For EMF-based editors, the properties view is a principle focal point when editing a model. Here is how things used to look:

image

This is how it looks now:

image

It's hard to tell what really has focus because there's always a blue under/over-line. The underlining of the tabbed page in the eidtor is different from the rest of the underlining, and changes color depending on focus. The breadcrumbs have some white stuff in them that I never seen before. Where did that come from?

Ranting constructively, this is one of the things that drive me to despair with theming. I might set a color, but something else will change that color and in the end I have no clue of how I can set colors such that all these different themes won't produce something completely unreadable in some cases. Basically one has to test every editor, view, and dialog on every theme, and even then I'm often left wondering, how do I fix this?

@merks: Can you pls. verify again with today's version of the "Light (Preview)" theme? Are the issues still there?

jukzi commented 1 month ago

Type hierarchy: same error: image