Change review status icons on PR view style to Github style (#10737)

* change the icon of ApproveReview pr from "eye" to "check" like github
* change the icon of RejectReview pr from "x" to "request-change" like github
* add "-" after "{{" which need to be one line (TODO: may be not change all)

Signed-off-by: a1012112796 <[email protected]>

Co-authored-by: Lunny Xiao <[email protected]>
Co-authored-by: John Olheiser <[email protected]>
Co-authored-by: guillep2k <[email protected]>

Author: 4 people

models/review.go

@@ -33,10 +33,10 @@ const (
 func (rt ReviewType) Icon() string {
 	switch rt {
 	case ReviewTypeApprove:
-		return "eye"
+		return "check"
 	case ReviewTypeReject:
-		return "x"
-	case ReviewTypeComment, ReviewTypeUnknown:
+		return "request-changes"
+	case ReviewTypeComment:
 		return "comment"
 	default:
 		return "comment"

models/review_test.go

@@ -44,8 +44,8 @@ func TestReview_LoadCodeComments(t *testing.T) {
 }
 
 func TestReviewType_Icon(t *testing.T) {
-	assert.Equal(t, "eye", ReviewTypeApprove.Icon())
-	assert.Equal(t, "x", ReviewTypeReject.Icon())
+	assert.Equal(t, "check", ReviewTypeApprove.Icon())
+	assert.Equal(t, "request-changes", ReviewTypeReject.Icon())
 	assert.Equal(t, "comment", ReviewTypeComment.Icon())
 	assert.Equal(t, "comment", ReviewTypeUnknown.Icon())
 	assert.Equal(t, "comment", ReviewType(4).Icon())

templates/repo/issue/view_content/comments.tmpl

@@ -318,7 +318,10 @@
 		</div>
 	{{else if eq .Type 22}}
 		<div class="event" id="{{.HashTag}}">
-			<span class="issue-symbol">{{svg (printf "octicon-%s" .Review.Type.Icon) 16}}</span>
+			<span class="issue-symbol {{if eq .Review.Type 1}}green
+			{{- else if eq .Review.Type 2}}grey
+			{{- else if eq .Review.Type 3}}red
+			{{- else}}grey{{end}}">{{svg (printf "octicon-%s" .Review.Type.Icon) 16}}</span>
 			{{if .OriginalAuthor }}
 			{{else}}
 				<a class="ui avatar image"{{if gt .Poster.ID 0}} href="{{.Poster.HomeLink}}"{{end}}>

templates/repo/issue/view_content/pull.tmpl

@@ -8,9 +8,9 @@
 					<div class="ui divider"></div>
 					<div class="review-item">
 						<span class="type-icon text {{if eq .Type 1}}green
-							{{else if eq .Type 2}}grey
-							{{else if eq .Type 3}}red
-							{{else}}grey{{end}}">
+							{{- else if eq .Type 2}}grey
+							{{- else if eq .Type 3}}red
+							{{- else}}grey{{end}}">
 							{{svg (printf "octicon-%s" .Type.Icon) 16}}
 						</span>
 						{{if .Stale}}
@@ -40,19 +40,19 @@
 {{end}}
 <div class="comment merge box">
 	<a class="avatar text
-	{{if .Issue.PullRequest.HasMerged}}purple
-	{{else if .Issue.IsClosed}}grey
-	{{else if .IsPullWorkInProgress}}grey
-	{{else if .IsFilesConflicted}}grey
-	{{else if .IsPullRequestBroken}}red
-	{{else if .IsBlockedByApprovals}}red
-	{{else if .IsBlockedByRejection}}red
-	{{else if and .EnableStatusCheck (or .RequiredStatusCheckState.IsFailure .RequiredStatusCheckState.IsError)}}red
-	{{else if and .EnableStatusCheck (or .RequiredStatusCheckState.IsPending .RequiredStatusCheckState.IsWarning)}}yellow
-	{{else if and .RequireSigned (not .WillSign)}}}red
-	{{else if .Issue.PullRequest.IsChecking}}yellow
-	{{else if .Issue.PullRequest.CanAutoMerge}}green
-	{{else}}red{{end}}">{{svg "octicon-git-merge" 32}}</a>
+	{{- if .Issue.PullRequest.HasMerged}}purple
+	{{- else if .Issue.IsClosed}}grey
+	{{- else if .IsPullWorkInProgress}}grey
+	{{- else if .IsFilesConflicted}}grey
+	{{- else if .IsPullRequestBroken}}red
+	{{- else if .IsBlockedByApprovals}}red
+	{{- else if .IsBlockedByRejection}}red
+	{{- else if and .EnableStatusCheck (or .RequiredStatusCheckState.IsFailure .RequiredStatusCheckState.IsError)}}red
+	{{- else if and .EnableStatusCheck (or .RequiredStatusCheckState.IsPending .RequiredStatusCheckState.IsWarning)}}yellow
+	{{- else if and .RequireSigned (not .WillSign)}}}red
+	{{- else if .Issue.PullRequest.IsChecking}}yellow
+	{{- else if .Issue.PullRequest.CanAutoMerge}}green
+	{{- else}}red{{end}}">{{svg "octicon-git-merge" 32}}</a>
 	<div class="content">
 		{{template "repo/pulls/status" .}}
 		<div class="ui {{if not $.LatestCommitStatus}}top attached header{{else}}attached merge-section segment{{end}}">