Skip to content

Conversation

@mvm-sap
Copy link
Contributor

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

The background color of bread-crumb items did not match the background color in the debug view, and this has been fixed with this PR. Find the before and after screen shots below for the same.

Before:
Screenshot 2025-01-22 125537

After:
Screenshot 2025-01-22 125326

@BeckerWdf BeckerWdf added this to the 4.35 M2 milestone Jan 22, 2025
@BeckerWdf
Copy link
Member

Does this PR supersede eclipse-platform/eclipse.platform.ui#2656?

@BeckerWdf
Copy link
Member

Is a similar fix also needed for the dark theme?

@mvm-sap
Copy link
Contributor Author

mvm-sap commented Jan 22, 2025

Does this PR supersede eclipse-platform/eclipse.platform.ui#2656?

Yes, We have to abandon that PR

@mvm-sap
Copy link
Contributor Author

mvm-sap commented Jan 22, 2025

Is a similar fix also needed for the dark theme?

Yes, I will do similar changes and create one more PR for dark theme

@github-actions
Copy link
Contributor

Test Results

 1 755 files  ±0   1 755 suites  ±0   1h 31m 9s ⏱️ + 1m 0s
 4 170 tests ±0   4 147 ✅ +1   23 💤 ±0  0 ❌  - 1 
13 107 runs  ±0  12 940 ✅ +1  167 💤 ±0  0 ❌  - 1 

Results for commit a38c068. ± Comparison against base commit e64b341.

@BeckerWdf
Copy link
Member

Tested on macOS. Looks good there too.

@BeckerWdf BeckerWdf merged commit 47197fc into eclipse-platform:master Jan 22, 2025
17 checks passed
@BeckerWdf
Copy link
Member

Thanks for fixing

@mvm-sap mvm-sap deleted the breadcrumb_fix_in_debug_view branch January 22, 2025 10:09
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