Approving smallish patch requires too many clicks



2 years ago
2 years ago


(Reporter: jimm, Unassigned)


(Depends on: 1 bug, Blocks: 1 bug)




2 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.

Comment 2

2 years ago
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


2 years ago
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).


2 years ago
Duplicate of this bug: 1263745

Comment 5

2 years ago
This will probably be fixed at the same time as bug 1207127.
Blocks: 1246611


2 years ago
Depends on: 1278958
You need to log in before you can comment on or make changes to this bug.