diff --git a/web-ui/src/components/reviews/TeamReviews.jsx b/web-ui/src/components/reviews/TeamReviews.jsx index d1fa34b88..b8cf9058d 100644 --- a/web-ui/src/components/reviews/TeamReviews.jsx +++ b/web-ui/src/components/reviews/TeamReviews.jsx @@ -782,13 +782,17 @@ const TeamReviews = ({ onBack, periodId }) => { }; const renderReviewer = (member, reviewer) => { + const hasReviewer = !!reviewer.name; const backgroundColor = reviewer.approved ? 'var(--checkins-palette-action-green)' : - 'var(--checkins-palette-action-yellow)'; + (hasReviewer ? + 'var(--checkins-palette-action-yellow)' : + 'var(--checkins-palette-action-red)'); const request = getReviewRequest(member, reviewer); const selfReviewRequest = getSelfReviewRequest(member); const variant = 'outlined'; - const statusLabel = `${reviewer.name}: ${getReviewStatus(request)}`; + const reviewerName = reviewer.name ?? "No Reviewer"; + const statusLabel = `${reviewerName}: ${getReviewStatus(request)}`; const url = getReviewerURL(request, selfReviewRequest); return (url ? @@ -802,9 +806,9 @@ const TeamReviews = ({ onBack, periodId }) => { : deleteReviewer(member, reviewer) : null} style={{backgroundColor: backgroundColor}} />); diff --git a/web-ui/src/styles/variables.css b/web-ui/src/styles/variables.css index 2f51a3da0..c62f2932d 100644 --- a/web-ui/src/styles/variables.css +++ b/web-ui/src/styles/variables.css @@ -66,6 +66,7 @@ --checkins-palette-action-disabled: var(--action-disabled); --checkins-palette-action-yellow: lightyellow; --checkins-palette-action-green: lightgreen; + --checkins-palette-action-red: darkred; &[data-mui-color-scheme='dark'] { --primary-text: color-mix(in oklab, var(--white), var(--black) 10%);