Skip to content

Conversation

@mvm-sap
Copy link
Contributor

@mvm-sap mvm-sap commented Dec 10, 2024

After improving the existing dark theme #2548, the Git staging view had side effects regarding the background colors of the whole view and also the Unstaged Changes and Staged Changes section. This has been fixed with this change by applying those dark background changes only to the editor. This also fixes the bg color of tool items.

Before:
image

After:
image

@github-actions
Copy link
Contributor

github-actions bot commented Dec 10, 2024

Test Results

 1 821 files  +  607   1 821 suites  +607   1h 32m 49s ⏱️ + 28m 52s
 7 729 tests ±    0   7 499 ✅ +    3  228 💤  -   4  2 ❌ +2 
24 348 runs  +8 116  23 597 ✅ +7 881  749 💤 +234  2 ❌ +2 

For more details on these failures, see this check.

Results for commit 8f54610. ± Comparison against base commit bbebe70.

♻️ This comment has been updated with latest results.

After improving the existing dark theme, Git staging view had side
effects wrt the back ground colors of whole view and also the Unstaged
Changes and Staged Changes section. This has been fixed with this change
by applying those dark background changes only to editor.
@BeckerWdf BeckerWdf force-pushed the Dark_theme_git_staging_view_fix branch from b3497af to 8f54610 Compare December 11, 2024 12:52
@BeckerWdf
Copy link
Member

failed tests are unrelated

@BeckerWdf BeckerWdf merged commit 8bd169c into eclipse-platform:master Dec 11, 2024
15 of 17 checks passed
@BeckerWdf BeckerWdf added this to the 4.35 M1 milestone Dec 13, 2024
@mvm-sap mvm-sap deleted the Dark_theme_git_staging_view_fix branch December 24, 2024 04:57
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