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.

Bug 134701

Summary: hierarchical layout is ugly compared to 6.1
Product: uml Reporter: George Vasick <gvasick>
Component: General DiagramAssignee: issues@uml <issues>
Status: REOPENED ---    
Severity: blocker    
Priority: P3    
Version: 6.x   
Hardware: All   
OS: All   
Issue Type: DEFECT Exception Reporter:
Bug Depends on: 153919    
Bug Blocks:    
Attachments: nb 6.1 layout
meteora layout
design pattern class diagram
default layout on cdfs
hierarchical layout
sample use case for different sized nodes

Description George Vasick 2008-05-07 22:25:09 UTC
See attached screenshots.
Comment 1 George Vasick 2008-05-07 22:26:14 UTC
Created attachment 61146 [details]
nb 6.1 layout
Comment 2 George Vasick 2008-05-07 22:26:56 UTC
Created attachment 61147 [details]
meteora layout
Comment 3 Sergey Petrov 2008-05-08 05:23:54 UTC
it may be possible to have correct automatic router for orthogonal layout, but in 6.1 there was direct router+bending
points. it may have sense to look into this way realization.
Comment 4 Peter Lam 2008-06-11 23:04:39 UTC
Created attachment 62704 [details]
design pattern class diagram
Comment 5 Peter Lam 2008-06-11 23:06:33 UTC
just attached the class diagram generated by applying a design pattern. Looks very ugly.
It's the "Chain Of Responsibility" design pattern from GoF.
Comment 6 George Vasick 2008-06-30 21:43:52 UTC
New layout algorithm available in the graph library now.
Comment 7 Peter Lam 2008-07-17 00:53:53 UTC
verified in buld 20080716. The layout is still very ugly. Attached is a new screenshot for the class diagram generated
by selected the bankpack and all its elements from the UML sample app.
1st screenshot is the default layout for cdfs of the bankpack and all its elements.
2nd screenshot is the result after clicking on the Hierarchical Layout button.
Comment 8 Peter Lam 2008-07-17 00:54:28 UTC
Created attachment 64777 [details]
default layout on cdfs
Comment 9 Peter Lam 2008-07-17 00:54:54 UTC
Created attachment 64778 [details]
hierarchical layout
Comment 10 Sergey Petrov 2008-09-24 12:55:57 UTC
Created attachment 70442 [details]
sample use case for different sized nodes
Comment 11 Sergey Petrov 2008-09-24 12:57:00 UTC
layout isn't goos even for simple cases like 3 node connected if nodes have different sizes, see attached scrrensot above.
Comment 12 Sergey Petrov 2008-10-02 18:38:00 UTC
what's the problem. absence of good layout affect our reverse engineering use case. in most cases user may want to re
and then create relatively big class diagram with all or some classes from this project.
Comment 13 Sergey Petrov 2008-11-26 09:21:23 UTC
*** Issue 153858 has been marked as a duplicate of this issue. ***
Comment 14 mharo 2008-11-28 07:14:15 UTC
The reverse engineering quality in NB6.1 is one of the major advantages in Netbeans over Eclipse. Because of this bug 
in release 6.5, we now need to have two versions of NB installed on our workstations. I hope someone finds the time 
and motivation to fix this asap.
Comment 15 Sergey Petrov 2008-12-05 15:37:29 UTC
*** Issue 154759 has been marked as a duplicate of this issue. ***
Comment 16 Quality Engineering 2008-12-22 15:10:43 UTC
Integrated into 'main-golden', will be available in build *200812221122* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/32b607aeddc4
User: sergeyp@netbeans.org
Log: fix #134701 - part fix, differently sized nodes are aligned correctly now (see last attached image)
Comment 17 Quality Engineering 2008-12-24 16:55:20 UTC
Integrated into 'main-golden', will be available in build *200812241401* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main/rev/67a5bad59e0d
User: sergeyp@netbeans.org
Log: fix #134701 - removed some unused code, may be related to issue fix later as one of integrations
Comment 18 Sergey Petrov 2009-01-26 19:07:25 UTC
it seems only minor improvement will be available in 7.0, target for next