Skip to content
This repository was archived by the owner on May 30, 2024. It is now read-only.

Commit be5263f

Browse files
author
Noah Lee
authored
Remove spinning for status (#304)
1 parent 2828f4e commit be5263f

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

ui/src/components/ReviewerList.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,13 @@ function ReviewItem(props: {review: Review}): JSX.Element {
2626
const status = (status: ReviewStatusEnum) => {
2727
switch (status) {
2828
case ReviewStatusEnum.Pending:
29-
return <SyncOutlined spin/>
29+
return <SyncOutlined />
3030
case ReviewStatusEnum.Approved:
3131
return <CheckOutlined style={{color: "green"}} />
3232
case ReviewStatusEnum.Rejected:
3333
return <CloseOutlined style={{color: "red"}} />
3434
default:
35-
return <SyncOutlined spin/>
35+
return <SyncOutlined />
3636
}
3737
}
3838

ui/src/components/StatusStateIcon.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ function mapStateToIcon(state: StatusState): JSX.Element {
6666
case StatusState.Null:
6767
return <></>
6868
case StatusState.Pending:
69-
return <SyncOutlined spin/>
69+
return <SyncOutlined />
7070
case StatusState.Success:
7171
return <CheckOutlined style={{color: colorSuccess}}/>
7272
case StatusState.Failure:
@@ -76,7 +76,7 @@ function mapStateToIcon(state: StatusState): JSX.Element {
7676
case StatusState.Skipped:
7777
return <StopOutlined />
7878
default:
79-
return <SyncOutlined spin/>
79+
return <SyncOutlined />
8080
}
8181
}
8282

0 commit comments

Comments
 (0)