thanks for this! I like the changes but the UI needs to be a bit more

[JenniferGuimond]

thanks for this! I like the changes but the UI needs to be a bit more polished, maybe have it default off for now just to get it merged in and then we can polish up the UI a bit.

On Sun, Apr 23, 2017 at 3:46 PM, Adnan Ali [email protected] wrote:

Hi @SamSaffron https://github.com/SamSaffron

I've made a few changes (demo in video link: https://drive.google.com/open?id=0B-8YziZr08bQaEF3VmNIQURFMWM ):

1. Added the ability to color by gem or method via radio buttons, without having to refresh the page.
2. Filter by width of frame (to get quick visibility on slower calls).
3. Ability to see callee in addition of the backtrace in the modal.
4. Zoom via click, instead of having to click and drag.

Would love your thoughts.

Additionally I'm working on getting filtering via method/gem, as well.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/SamSaffron/flamegraph/issues/23, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAUXXnYxibABWlxD5vFc8eDw2tT9zDOks5ry9TfgaJpZM4NFkFs .

Originally posted by @SamSaffron in https://github.com/SamSaffron/flamegraph/issues/23#issuecomment-297094391