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

[Dark Theme] Fix for bg color of bread crumb in editor #1844

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

mvm-sap
Copy link
Contributor

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

As we changed the background color of the editor to a darker shade, the background of the breadcrumbs was still using the older color code. This has been changed to the same color code as the editor's.

Before:
image

After:
image

@BeckerWdf
Copy link
Contributor

@jukzi: Can you pls. review and merge?

@jukzi jukzi force-pushed the fix_breadcrumb_dark_theme branch from 530f416 to 45f01ea Compare December 11, 2024 12:56
@jukzi
Copy link
Contributor

jukzi commented Dec 11, 2024

Can you pls. review and merge?

We refuse mere until CI is green again. You can help by solving underlying errors in platform.ui

@BeckerWdf
Copy link
Contributor

underlying errors in platform.ui

What is the underying issue in platform?

@jukzi
Copy link
Contributor

jukzi commented Dec 11, 2024

@BeckerWdf
Copy link
Contributor

BeckerWdf commented Dec 11, 2024

I see this in the "Tests / Build":

AutomatedSuite SmokeViewsTest.testOpenSourceView

This is an JDT test that fails. So I don't see that this is a platform issue. Or do I overlook something?

@jukzi
Copy link
Contributor

jukzi commented Dec 11, 2024

testOpenSourceView

see #1843 errors in platform pop up in jdt,

@BeckerWdf
Copy link
Contributor

testOpenSourceView

see #1843 errors in platform pop up in jdt,

So I see that a fix is on it's way with: eclipse-platform/eclipse.platform.ui#2612

@jukzi
Copy link
Contributor

jukzi commented Dec 11, 2024

So I see that a fix is on it's way with: eclipse-platform/eclipse.platform.ui#2612

yey, so please help to get that one green.

As we changed the background color of editor to more darker shade,
background of breadcrumbs were still using the older color code. This
has been changed to same color code as editor's.
@jukzi jukzi force-pushed the fix_breadcrumb_dark_theme branch from 45f01ea to 38e830c Compare December 16, 2024 12:43
@jukzi jukzi merged commit 44ef91f into eclipse-jdt:master Dec 16, 2024
10 checks passed
@BeckerWdf
Copy link
Contributor

Thanks for merging

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.

3 participants