Add Reviewer name to sidebar Reviewers list
What does this MR do and why?
For #329330 (closed)
This MR duplicates the "Assignees" behavior for un-collapsed assignees by:
- Adding the user's defined name, and
- Increasing the size of the user's avatar to match the extra height
Notable exclusions
Please note that this MR explicitly does not include the Reviewers' Busy status.
While it is likely useful to see that information, the Reviewer-type user
provided to the Reviewers list lacks the availability
or status
fields that are included with the Assignee-type users provided to the Assignees list.
Screenshots or screen recordings
Before | After |
---|---|
How to set up and validate locally
- Create an MR
- Assign a reviewer to it
- Gaze in awe upon the reviewer's name
- (optional) log in as the reviewer
- (optional) approve the MR as the reviewer
- (not optional) gaze in awe upon the green circle with the white check mark in it.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Thomas Randolph