[GitHub] [incubator-echarts] ryancui- edited a comment on pull request #12590: feat(graph): graph support multiple edges, for #6811

[GitBox <gi...@apache.org>]

ryancui- edited a comment on pull request #12590:
URL: https://github.com/apache/incubator-echarts/pull/12590#issuecomment-685529078


   I found that in some cases, toggle legend will make multiple lines into single one.(overlapping)
   
    Check it out in this demo: https://gallery.echartsjs.com/editor.html?c=xNQsu5DekH&v=2
   
   Then I figured that `edgeArray` is `undefined` after selecting legend, maybe caused by incorrect `node.dataIndex`.
   
   See here: https://github.com/wf123537200/incubator-echarts/blob/fd25c1a49d72cc32930c9a0836698d1a125b3298/src/chart/helper/multipleGraphEdgeHelper.js#L75
   
   I changed two lines, and it worked.
   
   ```js
   // var source = [n1.id, n1.dataIndex].join('.');
   var source = n1.id;
   // var target = [n2.id, n2.dataIndex].join('.');
   var target = n2.id;
   ```


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org