Closed
Bug 1153297
Opened 9 years ago
Closed 9 years ago
perfherder graphs should show the subtest summary when we have a highlighted revision
Categories
(Tree Management :: Perfherder, defect)
Tree Management
Perfherder
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: jmaher, Unassigned)
References
Details
right now for the subtest summary to display, we need to actively click on a dot on the graph. I would prefer that if the URL passed in highlightedrevision=<rev>, that we display the summary for that revision. issues: * what if the rev matches >1 series- which series do we display * need to watch out for non summary graphs if needed we could have a url param for displaySubTestSummary=true, or maybe the signature.
Comment 1•9 years ago
|
||
So, I understand the need for this. I would prefer to solve this by allowing the "disclosed" revision to be passed in as a separate url parameter, and then for the tooltip + summary to be automatically displayed. I don't think this should be very hard.
Comment 2•9 years ago
|
||
When I was implementing the revision highlighting I first thought of showing the subtest summary and opening up the tooltip but the problem was, as Joel mentioned in the case of more than one match for a revision, but I'm up for implementing wlach's suggestion.
Reporter | ||
Comment 3•9 years ago
|
||
sounds like a plan. The more flexible we can make this the better. I imagine another round of what makes sense after we get more things working. It opens the door to our creativity and points out what should be obvious :)
Reporter | ||
Comment 4•9 years ago
|
||
with the subtest summary being shifted to use compare mode, do we still think this is valid?
Comment 5•9 years ago
|
||
Let's WONTFIX this. I think we do still want the ability to persist a "selected" revision, but that's covered by bug 1153956.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•