Skip to content

Implement Cross-Platform Dark Theme Preferred API in SWT#3184

Merged
vogella merged 2 commits intoeclipse-platform:masterfrom
vogella:swt-dark-theme-api
Apr 8, 2026
Merged

Implement Cross-Platform Dark Theme Preferred API in SWT#3184
vogella merged 2 commits intoeclipse-platform:masterfrom
vogella:swt-dark-theme-api

Conversation

@vogella
Copy link
Copy Markdown
Contributor

@vogella vogella commented Apr 2, 2026

Implement Cross-Platform Dark Theme Preferred API in SWT. Will is currently handled in platform.ui and having such an API will make platform.ui easier and allows us to simplify the styling (and remove reflection access)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Test Results

  169 files   -     1    169 suites   - 1   23m 18s ⏱️ - 4m 22s
4 679 tests ±    0  4 658 ✅ ±  0   21 💤 ± 0  0 ❌ ±0 
5 579 runs   - 1 013  5 455 ✅  - 982  124 💤  - 31  0 ❌ ±0 

Results for commit 8cdfebe. ± Comparison against base commit cad012b.

♻️ This comment has been updated with latest results.

@vogella vogella force-pushed the swt-dark-theme-api branch 2 times, most recently from 8b7cb5b to 321386d Compare April 7, 2026 15:06
@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Apr 7, 2026

The warning is about the version filter at lines 246-252. Since the PR adds new APIs (dark theme), the version bump to 3.134.0 is now justified, making the filter unnecessary.

vogella and others added 2 commits April 8, 2026 09:29
The filter suppressing the minor version bump warning is no longer
needed since new APIs have been added in this release.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vogella vogella force-pushed the swt-dark-theme-api branch from eb81791 to 8cdfebe Compare April 8, 2026 07:29
@vogella vogella merged commit 9099f6c into eclipse-platform:master Apr 8, 2026
31 of 32 checks passed
@vogella vogella deleted the swt-dark-theme-api branch April 8, 2026 09:49
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