Closed
Bug 1499319
Opened 6 years ago
Closed 4 years ago
Allow for setting states from inside Perfherder popup
Categories
(Tree Management :: Perfherder, enhancement, P3)
Tree Management
Perfherder
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: igoldan, Unassigned)
Details
Attachments
(1 file)
By states we mean that it would be nice to add in "infra", "test change" - then we could be tricky with the graphs and ignore infra/test changes to show the real browser performance over time.
Comment 1•6 years ago
|
||
Similar to what the devtools team does in their graphs.
Comment 2•6 years ago
|
||
Reporter | ||
Updated•6 years ago
|
Attachment #9018178 -
Attachment is patch: true
Attachment #9018178 -
Attachment mime type: text/x-github-pull-request → text/plain
Attachment #9018178 -
Flags: feedback?(jmaher)
Updated•6 years ago
|
Attachment #9018178 -
Flags: feedback?(jmaher)
Reporter | ||
Comment 3•6 years ago
|
||
Given the upcoming work for switching Perfherder to React, this bug has a lower priority.
Reporter | ||
Updated•6 years ago
|
Priority: P1 → P2
Reporter | ||
Comment 4•6 years ago
|
||
From yesterday's discussion with :wlach, we concluded to avoid using the crosshair approach for selecting downstreams or reassigned alerts. Instead, we will employ popups very similar to those we use in the alert summary view.
Reporter | ||
Comment 5•6 years ago
|
||
Also, to avoid cluttering the black popups (when you click on data point), we will use a single carret (either right or left) that will expand it with the "Mark as" buttons.
Reporter | ||
Updated•5 years ago
|
Assignee: igoldan → nobody
Priority: P2 → P3
Reporter | ||
Comment 6•4 years ago
|
||
I'm not sure this is of much use, with the exception of marking an alert as invalid
. Closing this for now; if any Perf sheriff has other opinions, we can reopen this.
Reporter | ||
Updated•4 years ago
|
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•