Closed
Bug 1120623
Opened 10 years ago
Closed 9 years ago
Make the flamegraph keyboard accessible
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect, P3)
DevTools
Performance Tools (Profiler/Timeline)
Tracking
(firefox47 fixed)
RESOLVED
FIXED
Firefox 47
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: vporof, Assigned: vporof)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
jsantell
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
What kind of interaction do we want here? Arrow keys for scrolling, +/- for zoom? Or implement a cursor/playhead that seeks across the graph?
Assignee | ||
Updated•10 years ago
|
Blocks: perf-tool-v2
Comment 2•10 years ago
|
||
I'm using W/S for zoom and A/D for plan left and right. W/S can also be used for panning and zooming can be left to the mouse well instead.
To try it out:
http://people.mozilla.org/~bgirard/cleopatra/#report=9673aa6606a57f0af73fae9404bec85f4743e4fa
Hit 'Call Graph' in the middle-ish.
Assignee | ||
Updated•10 years ago
|
Comment 5•10 years ago
|
||
Is there a bug for making this mouse-accessible? Right now the only way to zoom in is the relatively coarse scroll / pan actions. It is common to expose vertical and horizontal slider UI to the user to a) indicate zooming is available and b) offer a more precise way to zoom.
Flags: needinfo?(vporof)
Flags: needinfo?(akratel)
Comment 6•10 years ago
|
||
No there's not, I believe -- do you mean scrollbars for pan/scroll (move left/right/up/down) or for zooming like a little scrubber on the side mapping to magnification (like Logic on OSX, can't find any images)?
Assignee | ||
Comment 7•10 years ago
|
||
No bug filed yet. What do you mean there's people not using trackpads? :)
Yeah, should make it better.
Flags: needinfo?(vporof)
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Priority: -- → P3
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(akratel)
Assignee | ||
Comment 8•10 years ago
|
||
Assignee | ||
Comment 9•9 years ago
|
||
How does this feel Jordan? Lemme know before I add any tests.
Attachment #8604724 -
Attachment is obsolete: true
Attachment #8642425 -
Flags: feedback?(jsantell)
Comment 10•9 years ago
|
||
Hmm.. so this feels right, I'd say maybe the left/right speed could be a bit faster (or maybe accelerate).
What's weird though is this is manipulating the selection, not the flame graph -- I'd assume I could also do this in other views, which I can't, and if we did apply them to there (as they're both abstract call tree views), they have their own keybindings, and we'd need a way to make sure they don't conflict (different key bindings? maybe have to give the details view or overview view focus?)
Also, side note, how can we convey all these hotkeys?
Updated•9 years ago
|
Attachment #8642425 -
Flags: feedback?(jsantell)
Comment 11•9 years ago
|
||
More info in bug 1159044 as well
Updated•9 years ago
|
Blocks: perf-tools-fx43
Updated•9 years ago
|
No longer blocks: perf-tools-fx43
Assignee | ||
Updated•9 years ago
|
Attachment #8642425 -
Attachment is obsolete: true
Assignee | ||
Comment 12•9 years ago
|
||
Rewrote this.
(In reply to Jordan Santell [:jsantell] [@jsantell] (Please needinfo) from comment #10)
> Hmm.. so this feels right, I'd say maybe the left/right speed could be a bit
> faster (or maybe accelerate).
Implemented acceleration for both zooming and panning.
>
> What's weird though is this is manipulating the selection, not the flame
> graph -- I'd assume I could also do this in other views, which I can't
The implementation does nothing to the "selection" as defined by the performance tool. In fact, the entire widget has no knowledge of the outside world. It's all contained inside the flame graph.
> Also, side note, how can we convey all these hotkeys?
Not sure, but I'd say that the arrow keys and WASD are pretty instinctual and don't need UI explaining them. You'd expect the DOWN/LEFT key to scroll in a document, same can be said in here.
Attachment #8715807 -
Flags: review?(jsantell)
Comment 13•9 years ago
|
||
Very slick. Code looks good, waiting for a build to finish to try this out
Comment 14•9 years ago
|
||
Comment on attachment 8715807 [details] [diff] [review]
flame-key-access.patch
Review of attachment 8715807 [details] [diff] [review]:
-----------------------------------------------------------------
Really smooth and nice. Wonder if we could have this for other views too, like waterfall? Would it make more sense for this to live in the overview graphs, like the mouse controls for it do? And the details view will use their own implementation (row selection on calltree/waterfall), otherwise falling back to the overview graph control (flamegraph)? Either way, can be dealt with in a follow up. Smooth like butter.
Attachment #8715807 -
Flags: review?(jsantell) → review+
Comment 16•9 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 47
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•