Skip to content

Commit 7769538

Browse files
committed
fix(revisions)/small fixes
1 parent aad288c commit 7769538

File tree

10 files changed

+31
-12
lines changed

10 files changed

+31
-12
lines changed

src/app/header/index.tsx

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,10 +66,20 @@ const Header = () => {
6666
onKeyDown={handleKeyDown}
6767
/>
6868
)}
69-
<a className="m-4 text-gray-600" href={GITHUB_MAIN}>
69+
<a
70+
className="m-4 text-gray-600"
71+
href={GITHUB_MAIN}
72+
target="_blank"
73+
rel="noopener noreferrer"
74+
>
7075
GitHub
7176
</a>
72-
<a className="m-4 text-gray-600" href={GITHUB_FEEDBACK}>
77+
<a
78+
className="m-4 text-gray-600"
79+
href={GITHUB_FEEDBACK}
80+
target="_blank"
81+
rel="noopener noreferrer"
82+
>
7383
Feedback
7484
</a>
7585
</div>

src/features/repo-list/index.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
margin-left: 38px;
33

44
&__placeholder {
5-
margin: 20px 0;
5+
margin: 30px 0;
66
text-align: center;
77
}
88

src/features/repo-list/index.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import React from "react";
2+
import { Empty } from "antd";
23
import { Repo, Tabs, SimplePagination, Card } from "shared/components";
34
import { str, dom } from "shared/helpers";
45
import { useReposQuery } from "./queries.gen";
@@ -53,6 +54,7 @@ const RepoList = ({ username }: Props) => {
5354
))
5455
) : (
5556
<h2 className="repo-list__placeholder">
57+
<Empty description={false} />
5658
{username} doesn’t have any repositories yet.
5759
</h2>
5860
)}

src/features/user-info/index.scss

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,11 +36,8 @@
3636
&__btn.follow,
3737
&__btn.unfollow,
3838
&__btn.edit {
39+
display: block;
3940
margin-top: 10px;
4041
transition: 0.4s;
41-
42-
&:hover {
43-
cursor: pointer;
44-
}
4542
}
4643
}

src/features/user-info/index.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@ const UserInfo = ({ username }: Props) => {
3232
<h1 className="user-info__name">{name}</h1>
3333
<h4 className="user-info__username">{username}</h4>
3434
<span className="user-info__bio">{bio}</span>
35-
<br></br>
3635
{isViewer ? (
3736
<Button className="user-info__btn edit" disabled title="While not available">
3837
Edit profile

src/pages/user/index.scss

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
.page-user {
2+
padding: 0 20px;
23
margin: 40px 10% !important;
34
}

src/shared/components/card/index.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
.card {
2-
height: 140px;
2+
height: 150px;
33
padding: 22px;
44
font-family: var(--ff-secondary);
55
font-size: 14px;
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
.card-skeleton {
22
width: 100% !important;
3-
height: 145px !important;
3+
height: 155px !important;
44
border-radius: 10px;
55
}

src/shared/components/simple-pagination/index.scss

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,14 @@
22
&.center {
33
text-align: center;
44
}
5+
6+
.prev-btn {
7+
border-top-right-radius: 0 !important;
8+
border-bottom-right-radius: 0 !important;
9+
}
10+
11+
.next-btn {
12+
border-top-left-radius: 0 !important;
13+
border-bottom-left-radius: 0 !important;
14+
}
515
}

src/shared/components/simple-pagination/index.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,10 @@ const SimplePagination = (props: Props) => {
2626
return (
2727
<div className={cn("simple-pagination", { center })}>
2828
<Button.Group>
29-
<Button disabled={!hasPrevPage} onClick={onPrev}>
29+
<Button className="prev-btn" disabled={!hasPrevPage} onClick={onPrev}>
3030
<LeftOutlined /> Prev
3131
</Button>
32-
<Button disabled={!hasNextPage} onClick={onNext}>
32+
<Button className="next-btn" disabled={!hasNextPage} onClick={onNext}>
3333
Next <RightOutlined />
3434
</Button>
3535
</Button.Group>

0 commit comments

Comments
 (0)