Approving smallish patch requires too many clicks



3 years ago
3 months ago


(Reporter: jimm, Unassigned)





3 years ago
It would be great if approving a patch could be accomplished from the page where I'm viewing the diff, without having to click through all the options and dialogs associated with r+ing a patch. Currently approval requires three clicks of the mouse -

- "finish review..." up top of the screen (hard to find and a bit ambiguous)
- check the "ship it" checkbox over on the left side of the screen
- click the publish button down at the bottom-right hand side of the screen

This is a rather annoying process to go through. A simple "Ship it" button in the diff view for single diff patches would be great.
Note that at least two times in the last three days I missed the second step of those, because of all the other steps I had to do... and then people were very confused at having no comments and the review request cancelled and trying to figure out what I meant.
We used to have a single "Ship It" button that would publish the review with a ship it. However, we removed it because people were clicking it and complaining that they didn't intend to grant r+. See bug 1195860 for what changed it.

Regarding the "Finish Review" dialog become a usability nightmare, there are several bugs on that and things are actively being worked on.
Depends on: 1195860
Product: Developer Services → MozReview
This has been bothering me a lot lately too. Chromium's Rietveld has this down to two clicks (and maybe I'm missing something): "Publish+Mail Comments", "Quick LGTM". There's also a keyboard shortcut for the first one (m).
Duplicate of this bug: 1263745
This will probably be fixed at the same time as bug 1207127.
Blocks: 1246611
Depends on: 1278958

Comment 6

3 months ago
MozReview is now obsolete. Please use Phabricator instead. Closing this bug.
Last Resolved: 3 months ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.