Skip to content

[GEF] Fully migrate to the GEF TreeViewer - #1432

Draft
ptziegler wants to merge 1 commit into
eclipse-windowbuilder:masterfrom
ptziegler:gef-tree-viewer
Draft

[GEF] Fully migrate to the GEF TreeViewer#1432
ptziegler wants to merge 1 commit into
eclipse-windowbuilder:masterfrom
ptziegler:gef-tree-viewer

Conversation

@ptziegler

Copy link
Copy Markdown
Contributor

This removes our copy of the TreeViewerEditPart and TreeViewer in favor of the upstream classes. In order to use the GEF viewer directly, a virtual DesignComponent edit part needs to be used as contents, holding the Tree widget. Otherwise the edit part of the first model is not shown.

@ptziegler ptziegler added this to the 1.24.0 milestone Apr 3, 2026
@github-actions

github-actions Bot commented Apr 3, 2026

Copy link
Copy Markdown

Test Results

     4 files  ±0       4 suites  ±0   2h 19m 18s ⏱️ + 6m 25s
 7 025 tests ±0   6 963 ✅  - 32  30 💤 ±0  32 ❌ +32 
23 696 runs  ±0  23 546 ✅  - 56  94 💤 ±0  56 ❌ +56 

For more details on these failures, see this check.

Results for commit f2ffa8b. ± Comparison against base commit 8e0799c.

♻️ This comment has been updated with latest results.

@ptziegler
ptziegler force-pushed the gef-tree-viewer branch 3 times, most recently from 25d857a to 957cef8 Compare April 3, 2026 22:27
@ptziegler ptziegler modified the milestones: 1.24.0, 1.25.0 Jun 16, 2026
@ptziegler ptziegler modified the milestones: 1.25.0, 1.26.0 Sep 3, 2026
ptziegler added a commit to ptziegler/windowbuilder that referenced this pull request Sep 5, 2026
In order to show the root `ObjectInfo` in the tree viewer, it must not
be used as input. For this, it is wrapped inside a `ObjectInfoContainer`
when passed to the tree viewer. This will then be transformed into a
`ObjectContainerEditPart`, which is the invisible child of the
`RootTreeEditPart`.

Contributes to
eclipse-windowbuilder#1432
ptziegler added a commit that referenced this pull request Sep 5, 2026
In order to show the root `ObjectInfo` in the tree viewer, it must not
be used as input. For this, it is wrapped inside a `ObjectInfoContainer`
when passed to the tree viewer. This will then be transformed into a
`ObjectContainerEditPart`, which is the invisible child of the
`RootTreeEditPart`.

Contributes to
#1432
This removes our copy of the `TreeViewer` and all of its references in
favor of the upstream class.
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