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

NEW
Unassigned

Status

MozReview
General
P4
normal
4 years ago
2 years ago

People

(Reporter: gps, Unassigned)

Tracking

Details

(Reporter)

Description

4 years ago
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.
(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
(Reporter)

Comment 2

4 years ago
P2 since bug 1034844 is also P2.
Depends on: 1034844
Priority: -- → P2

Comment 3

3 years ago
Less important with the new commit view but probably still useful.
Priority: P2 → P3

Updated

3 years ago
Depends on: 1135396

Comment 4

3 years ago
We have too many P1s, so I'm spreading out the priorities.  P3 -> P4, P2 -> P3, and some portion of P1s will become P2.
Priority: P3 → P4
(Assignee)

Updated

2 years ago
Product: Developer Services → MozReview
You need to log in before you can comment on or make changes to this bug.