Buttons in review box should indicate reason for disablement

[silverwind]

Description

When trying to review one's own PR, the UI buttons are disabled and there is no indication on why. I think a simple "You can not approve you own Pull Request" would go a long way in making it clear.

Screenshots

[silverwind]

Alternatively, we could just hide the disabled button and only show the comment button like GitHub does:

I'd say this is even better than disabling the buttons as the user is visually driven to the now-gray "Submit Review" button.

[jolheiser]

Alternatively, we could just hide the disabled button and only show the comment button like GitHub does:

I'd say this is even better than disabling the buttons as the user is visually driven to the now-gray "Submit Review" button.

Interesting, I still see the options, but they have tooltips for explanation

[silverwind]

Ah yes, the screenshot above was from a already-merged PR. Open PR shows the same UI as you see, so yes I think we should indicate with tooltip on both the red and green button.

[puni9869]

Could you assign this ticket to me.
This would be a good first issue for me. I have some experience with go and web development.