Skip to content

[GEF] Harmonize Source Feedback handling in EditPart with upstream class#1425

Merged
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:gef-source-feedback
Apr 2, 2026
Merged

[GEF] Harmonize Source Feedback handling in EditPart with upstream class#1425
ptziegler merged 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:gef-source-feedback

Conversation

@ptziegler
Copy link
Copy Markdown
Contributor

This is a continuation of 1ea6950. The eraseSourceFeedback(Request) and showSourceFeedback(Request) methods inside an EditPart should not check whether its EditPolicy understands the request; This should instead be done by the edit policies themselves.

@ptziegler ptziegler added this to the 1.24.0 milestone Apr 2, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026

Test Results

 2 720 files  ±0   2 720 suites  ±0   12h 35m 30s ⏱️ - 40m 32s
 7 018 tests ±0   6 960 ✅ ±0   58 💤 ±0  0 ❌ ±0 
28 072 runs  ±0  27 830 ✅ ±0  242 💤 ±0  0 ❌ ±0 

Results for commit 23a57ea. ± Comparison against base commit a177ac7.

♻️ This comment has been updated with latest results.

@ptziegler ptziegler force-pushed the gef-source-feedback branch from 3e59ee9 to d241407 Compare April 2, 2026 15:47
This is a continuation of 1ea6950. The
`eraseSourceFeedback(Request)` and `showSourceFeedback(Request)` methods
inside an `EditPart` should not check whether its `EditPolicy`
understands the request; This should instead be done by the edit
policies themselves.
@ptziegler ptziegler force-pushed the gef-source-feedback branch from d241407 to 23a57ea Compare April 2, 2026 17:24
@ptziegler ptziegler merged commit 2d82c8e into eclipse-windowbuilder:master Apr 2, 2026
8 checks passed
@ptziegler ptziegler deleted the gef-source-feedback branch April 2, 2026 22:11
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