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)

defect
Not set
normal

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.
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.
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.
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 :)
with the subtest summary being shifted to use compare mode, do we still think this is valid?
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.