search

surma / wasmphobia

https://wasmphobia.surma.technology/
Apache License 2.0
94 stars 5 forks source link

Push state for flame graph? #4

Open nickwesselman opened 4 months ago

nickwesselman commented 4 months ago

Is there a way to push state into the browser URL as you navigate the flame graph, so that when I accidentally hit back instead of Reset Zoom I don't exit the analysis entirely? 😅

surma commented 4 months ago

I am not generating the JavaScript that’s embedded in the flame graph. It currently uses replaceState instead of pushState (and I am not sure it’s listening to popstate events). But I’d agree that would be nice. I can definitely patch it in, but I’ll see if this may be worth upstreaming to inferno, too