search

sameeraroshan / visjs

visjs vaadin addon
Apache License 2.0
10 stars 13 forks source link

Node.setLevel(...) does not have any effect if the Node was already added #15

Closed syampillai closed 9 years ago

syampillai commented 9 years ago

In the following code, Node ("13") was added to the NeworkDiagram before setting its level using setLevel(2). However, it doesn't get displayed at Level 2.

    Options o = new Options();
    HierarchicalLayout h = new HierarchicalLayout();
    h.setEnabled(true);
    h.setDirection(Direction.UD);
    o.setHierarchicalLayout(h);
    NetworkDiagram nwd = new NetworkDiagram(o);
    Node n;
    n = new Node("00", "Root");
    n.setLevel(1);
    nwd.addNode(n);
    n = new Node("11", "One One");
    n.setLevel(2);
    nwd.addNode(n);
    n = new Node("12", "One Two");
    n.setLevel(2);
    nwd.addNode(n);
    n = new Node("13", "One Three");
    nwd.addNode(n);
    n.setLevel(2); // Problem here... Setting level to a Node that was already added, does not have any effect
    nwd.addEdge(new Edge("11", "00"));
    nwd.addEdge(new Edge("12", "00"));
    nwd.addEdge(new Edge("13", "00"));
sameeraroshan commented 9 years ago

well you have to update node to take effect.

On Fri, Jun 12, 2015 at 12:06 AM, syampillai notifications@github.com wrote:

In the following code, Node ("13") was added to the NeworkDiagram before setting its level using setLevel(2). However, it doesn't get displayed at Level 2.

Options o = new Options();
HierarchicalLayout h = new HierarchicalLayout();
h.setEnabled(true);

h.setDirection(Direction.UD); o.setHierarchicalLayout(h); NetworkDiagram nwd = new NetworkDiagram(o); Node n; n = new Node("00", "Root"); n.setLevel(1); nwd.addNode(n); n = new Node("11", "One One"); n.setLevel(2); nwd.addNode(n); n = new Node("12", "One Two"); n.setLevel(2); nwd.addNode(n); n = new Node("13", "One Three"); nwd.addNode(n); n.setLevel(2); // Problem here... Setting level to a Node that was already added, does not have any effect nwd.addEdge(new Edge("11", "00")); nwd.addEdge(new Edge("12", "00")); nwd.addEdge(new Edge("13", "00"));

— Reply to this email directly or view it on GitHub https://github.com/sameeraroshan/visjs/issues/15.

syampillai commented 9 years ago

Thanks Sameera, it works.