Skip to content

Conversation

@Vonna9
Copy link

@Vonna9 Vonna9 commented Oct 3, 2025

The following pull request addresses Issue #1858 - [manifest editor] Show uses of exported packages in a tree.

Changes:

  • Imported the tree viewer and tree content provider needed to start adding the tree to the ExportPackageVisibility.java file since this file along with the ExportPackageSection.java contains the code for displaying the export packages.

  • Worked on replacing some of the "table" functions with tree functions.

  • Added code that starts to build the tree for displaying export packages. I had technical difficulties committing this to this branch but will be add it to the code file

    **treeViewer = new TreeViewer(parent);
    treeViewer.setContentProvider(new TreeContentProvider());
    treeViewer.setLabelProvider(new TreeContentLabelProvider());

**I added this code where the main table display is coded in the ExportPackageVisibility.java file. I plan on adding more code to create the overall size of the tree later on. **

Questions:
Is there a specific tree display needed for this issue to show each "child of the package" or "used packages" as mentioned in the issue's description? For instance, creating a tree to display the parent package and a list of the child packages underneath it with checkboxes to see which packages are currently used:

Example of Export Package Display

I noticed that there is a file called "TreeSection.java" is it best practice to use this file and its code to help build the tree or is it specifically used for another feature in the Eclipse PDE project?

Author: Yvonna M.<[email protected]>
Date: Thu Oct 2 11:23:01 2025 -0700

Issue eclipse-pde#1815 - [manifest editor] Show uses of exported packages in a tree

Added the tree viewer needed to start adding the tree. Added code that starts to build the tree for displaying export packages.
@Vonna9 Vonna9 marked this pull request as ready for review October 7, 2025 17:55
@Vonna9 Vonna9 closed this Oct 7, 2025
@Vonna9 Vonna9 reopened this Oct 7, 2025
@Vonna9 Vonna9 marked this pull request as draft October 7, 2025 18:25
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