Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Light Theme]Fix for bg colors of entries in Progress View #2753

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mvm-sap
Copy link
Contributor

@mvm-sap mvm-sap commented Jan 23, 2025

Fix for issue #2313
Issue has been fixed in all three platforms Win, Mac and Linux.

@mvm-sap
Copy link
Contributor Author

mvm-sap commented Jan 23, 2025

@BeckerWdf Could you test once and confirm? Also, please update the screenshots as I have some issues uploading them.

Copy link
Contributor

Test Results

 1 818 files  ±0   1 818 suites  ±0   1h 35m 28s ⏱️ +44s
 7 716 tests ±0   7 488 ✅ ±0  228 💤 ±0  0 ❌ ±0 
24 309 runs  ±0  23 560 ✅ ±0  749 💤 ±0  0 ❌ ±0 

Results for commit ad6982b. ± Comparison against base commit f9a78c2.

@BeckerWdf
Copy link
Contributor

Tested on macOS:
It now looks like this:
image

But without your change it looks also good:
image

Can you pls. verify on your end?
Did the issue maybe only occur while the theme as the "Light (Preview)" theme?

@BeckerWdf
Copy link
Contributor

Pls. have a look at https://github.com/eclipse-platform/eclipse.platform.ui/blob/master/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/progress/ProgressInfoItem.java#L835

There the code checks for the current themeID. Depending on this some colors are set.
We should try to get rid of this color-setting logic and do this only in via the CSS files. But that would be a different issue / PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants