Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

RadialTreeLayout does not work properly #1017

Closed
seandenigris opened this issue Aug 3, 2015 · 5 comments
Closed

RadialTreeLayout does not work properly #1017

seandenigris opened this issue Aug 3, 2015 · 5 comments

Comments

@seandenigris
Copy link
Contributor

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
Copy link
Contributor Author 

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
Copy link
Contributor Author 

We will have a look at it.

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

@seandenigris
Copy link
Contributor Author 

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
Copy link
Contributor Author 

Roassal 1.499 fixes this. Thanks to Milton!

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

@seandenigris
Copy link
Contributor Author

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

  • Status changed: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@seandenigris