Adnan Ali
Adnan Ali
@SamSaffron I added the text in the graph back. Though I had to covert the rects to divs in order to make this happen while maintaining performance. This has a...
@SamSaffron One of the reasons I changed it to click to zoom is because I had found click and drag harder to navigate, specially on larger datasets. And had similar...
@SamSaffron Also, I changed the nodes to be html based in order to get the text in there, without compromising on speed. To get click and drag zoom to be...
@SamSaffron Is there something we can do halfway? Just been thinking about this. I see the benefit, of course, of being able to click and drag. It's just the speed...
@SamSaffron Sounds good. Let me see what I can do in the coming weeks. Thanks!
Hey @SamSaffron, One of the reasons I wasn't too keen on retaining the zooming functionality was because with a lot of nodes, it got super slow and hard to maintain....
@SamSaffron I've started to create smaller PRs on my fork: https://github.com/thisduck/flamegraph/pulls This way the changes can be small and merged in bit by bit into master.
@SamSaffron Have been plugging away and am very close to something sharable soon. When you get a moment, can you please send me an example of a flamegraph html that...
Thanks @jaydorsey. I'm going to be without a job in a couple of weeks and can give this attention so that there's either a toggle or some integrated version of...
So my timeline to work on this wasn't accurate. But am keen on doing something now. @jaydorsey Would you be able to share the html of some flamegraphs that you...