Merge requests > Approvals > Revoke approval button

After approving a merge request, the button turns into an orange warning button to "Revoke approval".

When you first click the approval button, the button is solid due to the hover state persisting:

Screen_Shot_2019-04-16_at_2.13.33_PM

Clicking away removes the hover state, showing that it is actually a secondary button:

Screen_Shot_2019-04-16_at_2.15.33_PM

Proposal

The button should not persist the hover state. Also, the orange warning button that appears makes me feel like I did something wrong. Since approvals can be easily toggled on/off, do we need to warn the user against revoking? Perhaps the button can be a default button.

Edited Apr 16, 2019 by Taurie Davis
Assignee Loading
Time tracking Loading