Skip to content

Commit a6d877d

Browse files
committed
fix pr icon
1 parent 6cb9d15 commit a6d877d

File tree

5 files changed

+19
-10
lines changed

5 files changed

+19
-10
lines changed

templates/repo/branch/list.tmpl

+1-1
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,7 @@
143143
{{if .LatestPullRequest.HasMerged}}
144144
<a href="{{.LatestPullRequest.Issue.Link}}" class="ui purple large label">{{svg "octicon-git-merge" 16 "tw-mr-1"}}{{ctx.Locale.Tr "repo.pulls.merged"}}</a>
145145
{{else if .LatestPullRequest.Issue.IsClosed}}
146-
<a href="{{.LatestPullRequest.Issue.Link}}" class="ui red large label">{{svg "octicon-git-pull-request" 16 "tw-mr-1"}}{{ctx.Locale.Tr "repo.issues.closed_title"}}</a>
146+
<a href="{{.LatestPullRequest.Issue.Link}}" class="ui red large label">{{svg "octicon-git-pull-request-closed" 16 "tw-mr-1"}}{{ctx.Locale.Tr "repo.issues.closed_title"}}</a>
147147
{{else}}
148148
<a href="{{.LatestPullRequest.Issue.Link}}" class="ui green large label">{{svg "octicon-git-pull-request" 16 "tw-mr-1"}}{{ctx.Locale.Tr "repo.issues.open_title"}}</a>
149149
{{end}}

templates/repo/issue/view_title.tmpl

+1-1
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@
4242
{{if .HasMerged}}
4343
<div class="ui purple label issue-state-label">{{svg "octicon-git-merge" 16 "tw-mr-1"}} {{if eq .Issue.PullRequest.Status 3}}{{ctx.Locale.Tr "repo.pulls.manually_merged"}}{{else}}{{ctx.Locale.Tr "repo.pulls.merged"}}{{end}}</div>
4444
{{else if .Issue.IsClosed}}
45-
<div class="ui red label issue-state-label">{{svg (Iif .Issue.IsPull "octicon-git-pull-request" "octicon-issue-closed")}} {{ctx.Locale.Tr "repo.issues.closed_title"}}</div>
45+
<div class="ui red label issue-state-label">{{svg (Iif .Issue.IsPull "octicon-git-pull-request-closed" "octicon-issue-closed")}} {{ctx.Locale.Tr "repo.issues.closed_title"}}</div>
4646
{{else if .Issue.IsPull}}
4747
{{if .IsPullWorkInProgress}}
4848
<div class="ui grey label issue-state-label">{{svg "octicon-git-pull-request-draft"}} {{ctx.Locale.Tr "repo.issues.draft_title"}}</div>

templates/shared/issueicon.tmpl

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
{{- if .PullRequest.HasMerged -}}
77
{{- svg "octicon-git-merge" 16 "text purple" -}}
88
{{- else -}}
9-
{{- svg "octicon-git-pull-request" 16 "text red" -}}
9+
{{- svg "octicon-git-pull-request-closed" 16 "text red" -}}
1010
{{- end -}}
1111
{{- else -}}
1212
{{- if .PullRequest.IsWorkInProgress ctx -}}

web_src/js/features/issue.ts

+14-7
Original file line numberDiff line numberDiff line change
@@ -3,28 +3,35 @@ import type {Issue} from '../types.ts';
33
export function getIssueIcon(issue: Issue) {
44
if (issue.pull_request) {
55
if (issue.state === 'open') {
6-
if (issue.pull_request.draft === true) {
6+
if (issue.pull_request.draft) {
77
return 'octicon-git-pull-request-draft'; // WIP PR
88
}
99
return 'octicon-git-pull-request'; // Open PR
10-
} else if (issue.pull_request.merged === true) {
10+
} else if (issue.pull_request.merged) {
1111
return 'octicon-git-merge'; // Merged PR
1212
}
13-
return 'octicon-git-pull-request'; // Closed PR
14-
} else if (issue.state === 'open') {
13+
return 'octicon-git-pull-request-closed'; // Closed PR
14+
}
15+
16+
if (issue.state === 'open') {
1517
return 'octicon-issue-opened'; // Open Issue
1618
}
1719
return 'octicon-issue-closed'; // Closed Issue
1820
}
1921

2022
export function getIssueColor(issue: Issue) {
2123
if (issue.pull_request) {
22-
if (issue.pull_request.draft === true) {
23-
return 'grey'; // WIP PR
24-
} else if (issue.pull_request.merged === true) {
24+
if (issue.state === 'open') {
25+
if (issue.pull_request.draft) {
26+
return 'grey'; // WIP PR
27+
}
28+
return 'green'; // Open PR
29+
} else if (issue.pull_request.merged) {
2530
return 'purple'; // Merged PR
2631
}
32+
return 'red'; // Closed PR
2733
}
34+
2835
if (issue.state === 'open') {
2936
return 'green'; // Open Issue
3037
}

web_src/js/svg.ts

+2
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ import octiconGitBranch from '../../public/assets/img/svg/octicon-git-branch.svg
3535
import octiconGitCommit from '../../public/assets/img/svg/octicon-git-commit.svg';
3636
import octiconGitMerge from '../../public/assets/img/svg/octicon-git-merge.svg';
3737
import octiconGitPullRequest from '../../public/assets/img/svg/octicon-git-pull-request.svg';
38+
import octiconGitPullRequestClosed from '../../public/assets/img/svg/octicon-git-pull-request-closed.svg';
3839
import octiconGitPullRequestDraft from '../../public/assets/img/svg/octicon-git-pull-request-draft.svg';
3940
import octiconGrabber from '../../public/assets/img/svg/octicon-grabber.svg';
4041
import octiconHeading from '../../public/assets/img/svg/octicon-heading.svg';
@@ -112,6 +113,7 @@ const svgs = {
112113
'octicon-git-commit': octiconGitCommit,
113114
'octicon-git-merge': octiconGitMerge,
114115
'octicon-git-pull-request': octiconGitPullRequest,
116+
'octicon-git-pull-request-closed': octiconGitPullRequestClosed,
115117
'octicon-git-pull-request-draft': octiconGitPullRequestDraft,
116118
'octicon-grabber': octiconGrabber,
117119
'octicon-heading': octiconHeading,

0 commit comments

Comments
 (0)