Clicking cancel after trying to remove an approver removes approver.

Summary

When you click 'remove' to remove someone from the approvers list for a merge request, you're prompted to confirm or cancel. If you click cancel, the approver is removed anyway.

Steps to reproduce

1. Make a merge request
2. Remove one of the approvers in the approver list
3. Click cancel when prompted

Expected behavior

Clicking cancel should not remove the approver. But even better would be to not prompt at all. Prompting isn't necessary.

Actual behavior

Clicking cancel removes the approver.

mentioned in issue #1742

mentioned in commit 0b63bb6e