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.

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.

1 participant