Closed Bug 1205258 Opened 9 years ago Closed 6 years ago

Presentation of review makes it very unclear which issues are still to be worked on

Categories

(MozReview Graveyard :: General, defect)

Product:
MozReview Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: jgraham, Unassigned)

Details

There's a little list at the top of open issues, but once you go to an issue or scroll down past that or something, it's very hard to find the actual actionable items amongst all the noise of issues that have already been resolved, and records of pushes, and so on.

In general I don't think the time-based list of changes makes sense for a review. What I'd like to see at the top is a) an indication of *what* has been reviewed (i.e. which pushes/files); this is a different bug and b) (this bug) what changes *I* need to make.

With both these changes, the review might look something like:

[header information about the commit and so on]

75% reviewed
Revision 1: 100%
Revision 2: 50%

2 open issues (hide)

  [diff]
  You can't spell "typo"

  [diff]
  Nuclear launch codes shouldn't be sent in plain text

17 resolved issues (show)
Product: Developer Services → MozReview
MozReview is now obsolete. Please use Phabricator instead. Closing this bug.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.