Torstein Hønsi

Results 233 comments of Torstein Hønsi
trafficstars

A general thought - actually this wasn't discussed in our planning doc (seems like none of us thought of it): We already have hierarchical series types, treemap and sunburst. These...

Thanks, getting close now! A couple of discussion points @pawelfus @pawellysy 1. The treegraph's cousins, treemap and sunburst, have a [levels option](https://api.highcharts.com/highcharts/series.sunburst.levels). This [doesn't work in the treegraph](https://jsfiddle.net/highcharts/8ph5d1Le/2/). Is it...

> Alternatively, we can add a collapsed state for markers I think that sounds a bit complicated - I mean the end user would have to look at the legend,...

> It's just the amount of possible configuration options that make me dizzy I know... Let's try to keep it simple. So we only need a `distance` from the node,...

Thanks, it's a good start! I'll come back with more detailed thoughts after releasing v10.0.1. For now I am thinking that we need some options to control whether these buttons...

Some more things to work on: 1. Since the treegraph now shares the data structure with treemap and sunburst, and each point has one or zero links attached to it,...

Interesting case... Yes I think it is acceptable at least for now. If it becomes a problem down the road we can discuss it then, perhaps introduce options for how...

@pawellysy A client also asked for `collapse` and `expand` events to be fired on the nodes, which I think is a very good idea.

@bre1470 Can you review this PR with your TypeScript glasses on? Some of it is refactoring, moving existing pieces of code around. The treegraph part is new. I have removed...

Apparently it is a [Chromium bug](https://bugs.chromium.org/p/chromium/issues/detail?id=1316315). The folks at DOMPurify are [having the same issue](https://github.com/cure53/DOMPurify/issues/529), and like them, I am reluctant to monkey patch the code for broken browsers. Is...