Skip to content

DO-NOT-MERGE-FOR-TESTING: Remove hard-coded button styling in workspace selection dialog#3918

Draft
vogella wants to merge 1 commit intoeclipse-platform:masterfrom
vogella:testing-unstyle-buttons-workspace-dialog
Draft

DO-NOT-MERGE-FOR-TESTING: Remove hard-coded button styling in workspace selection dialog#3918
vogella wants to merge 1 commit intoeclipse-platform:masterfrom
vogella:testing-unstyle-buttons-workspace-dialog

Conversation

@vogella
Copy link
Copy Markdown
Contributor

@vogella vogella commented Apr 21, 2026

Summary

Test PR for #3905.

The workspace selection dialog at IDE startup mimics dark-theme styling because the ThemeEngine is not yet available. On macOS this produces buttons that look different from the buttons on other dialogs (e.g. File > Switch Workspace).

This change skips the hard-coded dark styling for Button controls only, so they get the native macOS dark appearance. All other widgets (Shell, Composite, Label, Link, ExpandableComposite) continue to be styled as before.

Test plan

  • Start the IDE with a dark theme on macOS and confirm the workspace selection dialog buttons match the buttons in File > Switch Workspace.
  • Start the IDE with a light theme and confirm nothing changed.
  • Confirm the rest of the workspace selection dialog (background, labels, links, expandable settings section) still looks correct in dark mode.

…in workspace selection dialog

Skip the early dark-theme styling for Button controls in the workspace
selection dialog at startup so buttons get the native macOS dark
appearance instead, matching what the ThemeEngine applies later.

Other widgets (Shell, Composite, Label, Link, ExpandableComposite)
remain styled as before since the ThemeEngine is not yet available
during workspace selection.

Test change for eclipse-platform#3905
@github-actions
Copy link
Copy Markdown
Contributor

Test Results

   852 files  ±0     852 suites  ±0   57m 46s ⏱️ + 4m 4s
 7 911 tests ±0   7 668 ✅ ±0  243 💤 ±0  0 ❌ ±0 
20 235 runs  ±0  19 580 ✅ ±0  655 💤 ±0  0 ❌ ±0 

Results for commit 36d6113. ± Comparison against base commit 1cbae34.

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Apr 27, 2026

@BeckerWdf can you test this?

@BeckerWdf
Copy link
Copy Markdown
Member

@BeckerWdf can you test this?

Didn't I already last week?

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Apr 28, 2026

@BeckerWdf can you test this?

Didn't I already last week?

This one is for your remark here: #3905 (comment) and AFAICS you did not give me feedback on this PR (which might solve your button styling).

@BeckerWdf
Copy link
Copy Markdown
Member

Ah sorry. I did test it. This PR solves it. Somehow I did not write that last week.

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Apr 28, 2026

Ah sorry. I did test it. This PR solves it. Somehow I did not write that last week.

Can you post a screenshot, please?

@BeckerWdf
Copy link
Copy Markdown
Member

With your change:
image

Without your change:
image

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Apr 28, 2026

With your change: image

Without your change: image

That looks like the same to me? The new one is better? Can you explain why?

@BeckerWdf
Copy link
Copy Markdown
Member

With your change: image
Without your change: image

That looks like the same to me? The new one is better? Can you explain why?

Look at the buttons. They are very different. The first one is correct (as all buttons in the IDE look this way). The second one is wrong. I already highlighted the difference in #3905 (comment).

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Apr 28, 2026

I assume you expected a blue button like in the original example you posted. If the above is sufficient I start testing it on Linux and Windows. Windows testing usually takes a bit longer for me.

@BeckerWdf
Copy link
Copy Markdown
Member

I assume you expected a blue button like in the original example you posted. If the above is sufficient I start testing it on Linux and Windows. Windows testing usually takes a bit longer for me.

Sure. Having the button also in blue would be even better. But already this one is fetter then this strange "gradient" styled buttons.

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Apr 28, 2026

Linux Dark looks also really good:

image

Linux Light (unchanged to last years release):

image

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