Merge pull request #173 from ani-team/fix/test-revisions

fix(revisions)/small fixes

Author: AdeliyaG

src/app/header/index.tsx

@@ -66,10 +66,20 @@ const Header = () => {
                         onKeyDown={handleKeyDown}
                     />
                 )}
-                <a className="m-4 text-gray-600" href={GITHUB_MAIN}>
+                <a
+                    className="m-4 text-gray-600"
+                    href={GITHUB_MAIN}
+                    target="_blank"
+                    rel="noopener noreferrer"
+                >
                     GitHub
                 </a>
-                <a className="m-4 text-gray-600" href={GITHUB_FEEDBACK}>
+                <a
+                    className="m-4 text-gray-600"
+                    href={GITHUB_FEEDBACK}
+                    target="_blank"
+                    rel="noopener noreferrer"
+                >
                     Feedback
                 </a>
             </div>

src/features/repo-list/index.scss

@@ -2,7 +2,7 @@
     margin-left: 38px;
 
     &__placeholder {
-        margin: 20px 0;
+        margin: 30px 0;
         text-align: center;
     }
 

src/features/repo-list/index.tsx

@@ -1,4 +1,5 @@
 import React from "react";
+import { Empty } from "antd";
 import { Repo, Tabs, SimplePagination, Card } from "shared/components";
 import { str, dom } from "shared/helpers";
 import { useReposQuery } from "./queries.gen";
@@ -52,9 +53,10 @@ const RepoList = ({ username }: Props) => {
                         />
                     ))
                 ) : (
-                    <h2 className="repo-list__placeholder">
-                        {username} doesn’t have any repositories yet.
-                    </h2>
+                    <Empty
+                        className="repo-list__placeholder"
+                        description={<h2>{username} doesn’t have any repositories yet.</h2>}
+                    />
                 )}
             </div>
             <div className="repo-list__pagination mt-6">

src/features/user-info/index.scss

@@ -33,14 +33,9 @@
         line-height: 28px;
     }
 
-    &__btn.follow,
-    &__btn.unfollow,
-    &__btn.edit {
+    &__btn {
+        display: block !important;
         margin-top: 10px;
         transition: 0.4s;
-
-        &:hover {
-            cursor: pointer;
-        }
     }
 }

src/features/user-info/index.tsx

@@ -32,15 +32,14 @@ const UserInfo = ({ username }: Props) => {
             <h1 className="user-info__name">{name}</h1>
             <h4 className="user-info__username">{username}</h4>
             <span className="user-info__bio">{bio}</span>
-            <br></br>
             {isViewer ? (
-                <Button className="user-info__btn edit" disabled title="While not available">
+                <Button className="user-info__btn" disabled title="While not available">
                     Edit profile
                 </Button>
             ) : (
                 <Button
                     type={viewerIsFollowing ? "primary" : "default"}
-                    className={`user-info__btn ${label}`}
+                    className="user-info__btn"
                     onClick={() => following.handle(id, viewerIsFollowing)}
                     loading={following.debouncedLodaing}
                 >

src/pages/user/index.scss

@@ -1,3 +1,4 @@
 .page-user {
+    padding: 0 20px;
     margin: 40px 10% !important;
 }

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

@@ -2,4 +2,14 @@
     &.center {
         text-align: center;
     }
+
+    .prev-btn {
+        border-top-right-radius: 0 !important;
+        border-bottom-right-radius: 0 !important;
+    }
+
+    .next-btn {
+        border-top-left-radius: 0 !important;
+        border-bottom-left-radius: 0 !important;
+    }
 }

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

@@ -26,10 +26,10 @@ const SimplePagination = (props: Props) => {
     return (
         <div className={cn("simple-pagination", { center })}>
             <Button.Group>
-                <Button disabled={!hasPrevPage} onClick={onPrev}>
+                <Button className="prev-btn" disabled={!hasPrevPage} onClick={onPrev}>
                     <LeftOutlined /> Prev
                 </Button>
-                <Button disabled={!hasNextPage} onClick={onNext}>
+                <Button className="next-btn" disabled={!hasNextPage} onClick={onNext}>
                     Next <RightOutlined />
                 </Button>
             </Button.Group>