10BC0 Replace CCombo3 with native SWT CCombo by ptziegler · Pull Request #1092 · eclipse-windowbuilder/windowbuilder · GitHub
[go: up one dir, main page]

Skip to content

Replace CCombo3 with native SWT CCombo#1092

Merged
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:issue1090
Jan 14, 2026
Merged

Replace CCombo3 with native SWT CCombo#1092
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:issue1090

Conversation

@ptziegler
Copy link
Contributor

Both widgets behave effectively the same so there is no need for out custom implementation when the functionality is already provided.

Due to 5064677, the CCombo3 widget also doesn't work anymore, because unlike the CImageLabel, the CLabel always sets the SWT.NO_FOCUS bit which is causing issues with the drop-down table.

Closes #1090

@ptziegler
Copy link
Contributor Author
ptziegler 8000 commented May 23, 2025

Removing the widget alltogether is propably the better choice but that's too risky for the R1. Will be delayed for the M1 instead.

ptziegler added a commit to ptziegler/windowbuilder that referenced this pull request Jan 14, 2026
Both widgets behave effectively the same so there is no need for out
custom implementation when the functionality is already provided.

Due to 5064677, the CCombo3 widget also doesn't work anymore, because
unlike the CImageLabel, the CLabel always sets the SWT.NO_FOCUS bit
which is causing issues with the drop-down table.

Closes eclipse-windowbuilder#1090
ptziegler added a commit to ptziegler/windowbuilder that referenced this pull request Jan 14, 2026
Both widgets behave effectively the same so there is no need for out
custom implementation when the functionality is already provided.

Due to 5064677, the CCombo3 widget also doesn't work anymore, because
unlike the CImageLabel, the CLabel always sets the SWT.NO_FOCUS bit
which is causing issues with the drop-down table.

Closes eclipse-windowbuilder#1090
@github-actions
Copy link
github-actions bot commented Jan 14, 2026

Test Results

 2 720 files  ±0   2 720 suites  ±0   10h 36m 14s ⏱️ - 8m 43s
 7 004 tests ±0   6 946 ✅ ±0   58 💤 ±0  0 ❌ ±0 
28 016 runs  ±0  27 774 ✅ ±0  242 💤 ±0  0 ❌ ±0 

Results for commit 59613f5. ± Comparison against base commit b8f45d1.

♻️ This comment has been updated with latest results.

ptziegler added a commit to ptziegler/windowbuilder that referenced this pull request Jan 14, 2026
Both widgets behave effectively the same so there is no need for out
custom implementation when the functionality is already provided.

Due to 5064677, the CCombo3 widget also doesn't work anymore, because
unlike the CImageLabel, the CLabel always sets the SWT.NO_FOCUS bit
which is causing issues with the drop-down table.

Closes eclipse-windowbuilder#1090
Both widgets behave effectively the same so there is no need for out
custom implementation when the functionality is already provided.

Due to 5064677, the CCombo3 widget also doesn't work anymore, because
unlike the CImageLabel, the CLabel always sets the SWT.NO_FOCUS bit
which is causing issues with the drop-down table.

Closes eclipse-windowbuilder#1090
@ptziegler ptziegler added this to the 1.23.0 milestone Jan 14, 2026
@ptziegler ptziegler merged commit 6ee0cd8 into eclipse-windowbuilder:master Jan 14, 2026
8 checks passed
@ptziegler ptziegler deleted the issue1090 branch January 14, 2026 20:27
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.

Cannot change properties using option lists

1 participant

0