Closed
Bug 777673
Opened 12 years ago
Closed 12 years ago
next-/prev-unstarred shortcuts skip a failure after starring twice with onlyunstarred=1
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: mbrubeck)
References
Details
Attachments
(1 file)
2.66 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
1) Go to https://tbpl.mozilla.org/?tree=Mozilla-Inbound&onlyunstarred=1 2) Press 'k' 3) Star the first failure (click on the suggested bug star, then the quickstar button) 4) Press 'k' 5) Star this failure 6) Press 'k' Expected: After step #6, the first visible failure is selected. Actual: After step #6, the second visible failure is selected. (Works fine at step #4, just broken for the second starring) Must have not starred two in a row like this (without clicking), when I was testing bug 776692.
Assignee | ||
Comment 1•12 years ago
|
||
It looks like this happens when you have onlyunstarred=1 and the selected job is hidden because it is starred.
Summary: next-/prev-unstarred shortcuts skip a failure after starring twice → next-/prev-unstarred shortcuts skip a failure after starring twice with onlyunstarred=1
Assignee | ||
Comment 2•12 years ago
|
||
The next/prev-unstarred shortcut handler sorts the results by "result.order". This does not work correctly when the selected result is hidden, because _buildHTMLForOS sets .order only for displayed results. (For hidden results it will be uninitialized or set to an old incorrect value.) This patch fixes this by changing _buildHTMLForOS to set .order on *all* of the results, before filtering out the hidden ones. (Note: The "order" property is currently used only for these keyboard shortcuts, so this shouldn't affect anything else.)
Reporter | ||
Comment 3•12 years ago
|
||
Comment on attachment 646193 [details] [diff] [review] patch Looks good & works well locally for me :-)
Attachment #646193 -
Flags: review?(bmo) → review+
Assignee | ||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/ce4276d1d49c
Assignee | ||
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Webtools → Tree Management
Updated•9 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•