Closed
Bug 1173464
Opened 9 years ago
Closed 2 years ago
Markers should have tooltips in MarkerDetails to help explain the marker itself
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect, P3)
DevTools
Performance Tools (Profiler/Timeline)
Tracking
(firefox41 affected)
RESOLVED
INVALID
Tracking | Status | |
---|---|---|
firefox41 | --- | affected |
People
(Reporter: jsantell, Unassigned)
Details
We have a handful of markers that developers may be familiar with if they've used Chrome's devtools (recalc style, layout, paint), and additional ones that are either Firefox specific (Cycle Collection) or something Chrome tools does not have (Parse HTML?).
We should have a tooltip in the marker details view (probably a [?] maybe?) that gives a 1-2 sentence description of what this marker represents. Optionally a link to MDN with more information.
Comment 1•9 years ago
|
||
I don't think there is value in duplicating what the MDN docs should already say in a better fashion. We should just ensure that each marker type has its own anchor, and link there.
Additionally, we could pop open a Tooltip view with the MDN page instead of opening it in a new tab.
Reporter | ||
Comment 2•9 years ago
|
||
It's not duplicating -- one is a tooltip for a brief description, no different than tooltips describing the columns in call tree views, whereas the docs[0] can go into further detail.
If we want the tooltips to be like inspector's MDN Tooltips, that should be moderately straight forward, and OK as well if we want to display more than just "tooltip" info.
Also wondering if the MDN Tooltip can cache -- it uses nsIXMLHttpRequest now, which may handle caching on its own, but something to consider in offline modes.
[0] https://developer.mozilla.org/en-US/docs/Tools/Performance/Waterfall#Markers
Reporter | ||
Updated•9 years ago
|
Blocks: perf-tools-fx42
Reporter | ||
Updated•9 years ago
|
Reporter | ||
Updated•9 years ago
|
No longer blocks: perf-tools-fx43
Comment 3•9 years ago
|
||
Triaging. Filter on ADRENOCORTICOTROPIC (yes).
Priority: -- → P3
Version: 41 Branch → unspecified
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•2 years ago
|
Severity: normal → S3
Comment 4•2 years ago
|
||
This report is related to the old DevTools profiler implementation, which has been removed. The Performance panel is now based on Firefox Profiler at https://profiler.firefox.com/
Closing as invalid
Updated•2 years ago
|
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•