This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Summary: | Patch for: Display dependencies in a tree rather than a flat list (or graph) | ||
---|---|---|---|
Product: | projects | Reporter: | sgtgarcia <sgtgarcia> |
Component: | Maven | Assignee: | Milos Kleint <mkleint> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | cezariusz, markiewb |
Priority: | P3 | Keywords: | NETFIX, PATCH_AVAILABLE, UI |
Version: | 6.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | ENHANCEMENT | Exception Reporter: | |
Bug Depends on: | 235504 | ||
Bug Blocks: | 199852 | ||
Attachments: |
Proposed patch
Patch in action |
Description
sgtgarcia
2010-03-04 05:07:29 UTC
Reassigning to default owner. Doing this in the Projects view does not look like it would work well; subnodes of artifacts are already the JAR contents. But I can at least improve indirect dependencies to show their trail, in the tool tip. Integrated into 'main-golden' Changeset: http://hg.netbeans.org/main-golden/rev/a165889df1d0 User: Jesse Glick <jglick@netbeans.org> Log: Noted in #181517: display dependency trail (where nontrivial) in a tool tip. What about putting the dependency tree in the Artifact Detail view instead of a tooltip? The Artifact Detail window is used for artifacts, whereas only dependencies have a trail. (A dependency is an artifact, plus a trail, plus a scope, etc.) *** Bug 204681 has been marked as a duplicate of this bug. *** Possible NetFIX candidate. How about providing some more layouts? In DependencyGraphScene.cleanLayout() you could use several other layouts. <pre> //Variant a) Hierarchical GraphLayout GraphLayout layoutGraph = GraphLayoutFactory.createHierarchicalGraphLayout(this, true, false); // layoutGraph.layoutGraph(this); //Variant b) Tree GraphLayout vertical GraphLayout layoutTreeVertical = GraphLayoutFactory.createTreeGraphLayout(10, 10, 50, 50, true); GraphLayoutSupport.setTreeGraphLayoutRootNode(layoutTreeVertical, rootNode); // layoutTreeVertical.layoutGraph(this); //Variant c) Tree GraphLayout horizontal GraphLayout layoutTreeHorizontal = GraphLayoutFactory.createTreeGraphLayout(10, 10, 50, 50, false); GraphLayoutSupport.setTreeGraphLayoutRootNode(layoutTreeHorizontal, rootNode); // layoutTreeHorizontal.layoutGraph(this); //Variant d) default layout in 7.3 layout = new FruchtermanReingoldLayout(this, panel); layout.invokeLayout(); </pre> Created attachment 135908 [details]
Proposed patch
I like to propose a patch. The patch introduces popupmenu items which allow the user to switch the used layout algorithm. See screenshot.
@Milos: Please review and commit. (The patch is a candidate for post-NB74)
Created attachment 135909 [details]
Patch in action
layouts in graph is related to issue issue 235504 patch applied with minor finetuning (zoom to fit after changing layout and make all layouts animated) http://hg.netbeans.org/core-main/rev/aed91faf17d1 |