1083870 - Add next and previous links to move between reviews in a series

Status: RESOLVED INVALID

(MozReview Graveyard :: General, defect, P4)

Description

[Gregory Szorc [:gps]]

I think it would be rad if the review page for an individual review had "next" and "previous" links (UI TBD) that allowed you to easily move between reviews in the series. This would save a lot of time. Today, you have to go back to the main review page and find the appropriate review. Or, you have to load the commits tab and do the same.

Comment 1

[Steven MacLeod [:smacleod]]

(In reply to Gregory Szorc [:gps] from comment #0)
> I think it would be rad if the review page for an individual review had
> "next" and "previous" links (UI TBD) that allowed you to easily move between
> reviews in the series. This would save a lot of time. Today, you have to go
> back to the main review page and find the appropriate review. Or, you have
> to load the commits tab and do the same.

Totally Agree. Right now I deal with this by opening every commit in its own tab and moving that way. (Also, we need to get the Description field back inside the diff view page because if I want to read the commit message while I'm reviewing the diff I have to load the reviews page again).

Also, I reported something like this a while back, and proposed a couple solutions. See Bug 1034844

Comment 2

[Gregory Szorc [:gps]]

P2 since bug 1034844 is also P2.

Comment 3

[Mark Côté [:mcote]]

Less important with the new commit view but probably still useful.

Comment 4

[Mark Côté [:mcote]]

We have too many P1s, so I'm spreading out the priorities.  P3 -> P4, P2 -> P3, and some portion of P1s will become P2.

Comment 5

[BMO Automation]

MozReview is now obsolete. Please use Phabricator instead. Closing this bug.