moosetechnology / Moose

MOOSE - Platform for software and data analysis.
https://moosetechnology.github.io/moose-wiki
MIT License
136 stars 34 forks source link

RadialTreeLayout does not work properly #1017

Closed seandenigris closed 9 years ago

seandenigris commented 9 years ago

Originally reported on Google Code with ID 1017

Try this:

view := ROMondrianViewBuilder new.
view nodes: (1 to: 99).
view edgesFrom: [ :x | x // 10 ].
view radialTreeLayout.
view open

You will see only one tree. We should see ten.

See here:
view := ROMondrianViewBuilder new.
view nodes: (1 to: 99).
view edgesFrom: [ :x | x // 10 ].
view treeLayout.
view open

Reported by tudor@tudorgirba.com on 2013-11-27 13:19:20

seandenigris commented 9 years ago
On the other hand, NarrowRadialTreeLayout works fine.

view := ROMondrianViewBuilder new.
view nodes: (1 to: 99).
view edgesFrom: [ :x | x // 10 ].
view radialNarrowTreeLayout.
view open

Reported by tudor@tudorgirba.com on 2013-11-27 13:23:28

seandenigris commented 9 years ago
We will have a look at it.

Reported by alexandre.bergel@objectprofile.com on 2013-11-27 13:24:48

seandenigris commented 9 years ago
But, here is something even more fishy. When nested, RadialNarrowTreeLayout works in
a funny way:

view := ROMondrianViewBuilder new.
view node: #a forIt: [ 
    view nodes: (1 to: 99).
    view edgesFrom: [ :x | x // 10 ].
    view radialNarrowTreeLayout ].
view open

At the same time, the ragular TreeLayout works fine:
view := ROMondrianViewBuilder new.
view node: #a forIt: [ 
    view nodes: (1 to: 99).
    view edgesFrom: [ :x | x // 10 ].
    view treeLayout ].
view open

Reported by tudor@tudorgirba.com on 2013-11-27 13:25:21

seandenigris commented 9 years ago
Roassal 1.499 fixes this. Thanks to Milton!

Reported by alexandre.bergel@objectprofile.com on 2013-11-28 21:51:33

seandenigris commented 9 years ago

Reported by alexandre.bergel on 2013-11-28 21:52:14