fix: multiple bug fixes (#179)

* fix(Buttons): external icon not separated from text * feat(Navbar): change width to 100% for cleaner look * fix(Button): only add space for items with text and icon * fix: remove usage of deprecated functions
ijsKoud · Sep 4, 2022 · fef55ad · fef55ad
1 parent 813aa0b
commit fef55ad
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/src/components/general/Modal/index.tsx b/src/components/general/Modal/index.tsx
@@ -9,7 +9,7 @@ interface Props {
 }
 
 const Modal: FC<Props> = ({ children, onClick, isOpen }) => {
-	return <AnimatePresence exitBeforeEnter>{isOpen && <ModalItem onClick={onClick}>{children}</ModalItem>}</AnimatePresence>;
+	return <AnimatePresence mode="wait">{isOpen && <ModalItem onClick={onClick}>{children}</ModalItem>}</AnimatePresence>;
 };
 
 export default Modal;
diff --git a/src/pages/index.tsx b/src/pages/index.tsx
@@ -13,7 +13,7 @@ const Home: NextPage = () => {
 				</div>
 				<p>File uploading. URL Shortening. Protected views.</p>
 				<div className="home-buttons">
-					<Button external newWindow style="black" type="link" text="GitHub" url="https://ijskoud.dev/github/paperplane" />
+					<Button external newWindow style="black" type="link" text="GitHub " url="https://ijskoud.dev/github/paperplane" />
 					<Button style="black" type="link" text="Dashboard" url="/dashboard" />
 				</div>
 			</div>

diff --git a/src/styles/components/navbar.scss b/src/styles/components/navbar.scss
@@ -34,7 +34,7 @@
 .navbar-container-wrapper {
 	width: 100%;
 
-	top: 20px;
+	top: 0px;
 	left: 50%;
 	transform: translateX(-50%);
 
@@ -51,8 +51,7 @@
 	background-color: $navbar-background;
 	padding: 10px 20px;
 
-	border-radius: 10px;
-	width: 90%;
+	width: 100%;
 	height: 80px;
 
 	display: flex;