Skip to content

[GEF] Harmonize getTargetEditPart(...) in EditPart with upstream class#1413

Open
ptziegler wants to merge 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:gef-edit-part-target-ep
Open

[GEF] Harmonize getTargetEditPart(...) in EditPart with upstream class#1413
ptziegler wants to merge 1 commit intoeclipse-windowbuilder:masterfrom
ptziegler:gef-edit-part-target-ep

Conversation

@ptziegler
Copy link
Copy Markdown
Contributor

This is a continuation of 1be791a. The getTargetEditPart(Request) method 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 Mar 28, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 28, 2026

Test Results

 2 720 files  ±0   2 720 suites  ±0   10h 56m 46s ⏱️ + 4m 54s
 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 8fc73ba. ± Comparison against base commit 1be791a.

♻️ This comment has been updated with latest results.

@ptziegler ptziegler force-pushed the gef-edit-part-target-ep branch 3 times, most recently from eef5f5f to 1bc0c9e Compare March 28, 2026 19:31
This is a continuation of 1be791a. The
`getTargetEditPart(Request)` method 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-edit-part-target-ep branch from 1bc0c9e to 8fc73ba Compare March 28, 2026 20:15
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