From 2159fd37d590287e2e595053dac0dd45c9e73667 Mon Sep 17 00:00:00 2001
From: BFlat <bombayapirat@gmail.com>
Date: Sun, 24 Mar 2024 17:48:02 +0700
Subject: [PATCH] button size change

---
 app/components/Button.tsx           | 42 ++++++++++++++---------------
 app/components/ButtonLink.tsx       | 30 +++++++++++----------
 app/modules/TopBar/TopBar.tsx       |  2 +-
 app/pages/PageEnigma/PageEnigma.tsx | 10 +++----
 app/styles/base.css                 |  2 +-
 5 files changed, 42 insertions(+), 44 deletions(-)

diff --git a/app/components/Button.tsx b/app/components/Button.tsx
index 81c69ce7e4..87c9c7bde8 100644
--- a/app/components/Button.tsx
+++ b/app/components/Button.tsx
@@ -1,41 +1,39 @@
 import { IconDefinition } from "@fortawesome/fontawesome-svg-core";
 import { FontAwesomeIcon } from "@fortawesome/react-fontawesome";
 
-export interface ButtonPropsI extends React.ButtonHTMLAttributes<HTMLButtonElement> {
- icon?: IconDefinition;
- variant?: "primary"|"secondary";
+export interface ButtonPropsI
+  extends React.ButtonHTMLAttributes<HTMLButtonElement> {
+  icon?: IconDefinition;
+  variant?: "primary" | "secondary";
 }
 
 export const Button = ({
   icon,
   children,
-  className : propsClassName,
-  variant = 'primary',
+  className: propsClassName,
+  variant = "primary",
   ...rest
-}:ButtonPropsI)=>{
-  function getVariantClassNames(variant: string){
-    switch(variant){
-      case "secondary":{
+}: ButtonPropsI) => {
+  function getVariantClassNames(variant: string) {
+    switch (variant) {
+      case "secondary": {
         return " bg-brand-secondary hover:bg-brand-secondary-900 text-white focus-visible:outline-brand-secondary";
       }
       case "primary":
-      default:{
+      default: {
         return " bg-brand-primary hover:bg-brand-primary-400 text-white focus-visible:outline-brand-primary-600";
       }
     }
-  }; 
-  const className = "text-sm font-semibold rounded-md px-3.5 py-2.5 shadow-sm focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2" + getVariantClassNames(variant)
-  + (propsClassName ? ` ${propsClassName}` : "");
+  }
+  const className =
+    "text-sm font-semibold rounded-md px-3 py-2 shadow-sm focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 transition-all duration-150" +
+    getVariantClassNames(variant) +
+    (propsClassName ? ` ${propsClassName}` : "");
 
-  return(
-    <button
-      className={className}
-      {...rest}
-    >
-      {icon && 
-        <FontAwesomeIcon className="mr-2" icon={icon}/>
-      }
+  return (
+    <button className={className} {...rest}>
+      {icon && <FontAwesomeIcon className="mr-2" icon={icon} />}
       {children}
     </button>
   );
-}
\ No newline at end of file
+};
diff --git a/app/components/ButtonLink.tsx b/app/components/ButtonLink.tsx
index f7e6fcfdc9..2d6ad78391 100644
--- a/app/components/ButtonLink.tsx
+++ b/app/components/ButtonLink.tsx
@@ -4,38 +4,40 @@ import { FontAwesomeIcon } from "@fortawesome/react-fontawesome";
 
 interface LinkButtonProps extends LinkProps {
   icon?: IconDefinition;
-  variant?: "primary"|"secondary";
- }
+  variant?: "primary" | "secondary";
+}
 
- export const ButtonLink = ({
+export const ButtonLink = ({
   icon,
   children,
   className: propsClassName,
   variant = "primary",
   ...rest
- }:LinkButtonProps)=>{
+}: LinkButtonProps) => {
   //TODO: Duplicated from Button.tsx
-  function getVariantClassNames(variant: string){
-    switch(variant){
-      case "secondary":{
+  function getVariantClassNames(variant: string) {
+    switch (variant) {
+      case "secondary": {
         return " bg-brand-secondary hover:bg-brand-secondary-900 text-white focus-visible:outline-brand-secondary";
       }
       case "primary":
-      default:{
+      default: {
         return " bg-brand-primary hover:bg-brand-primary-400 text-white focus-visible:outline-brand-primary-600";
       }
     }
-  }; 
-  const className = "text-sm font-semibold rounded-md px-3.5 py-2.5 shadow-sm focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2" + getVariantClassNames(variant)
-  + (propsClassName ? ` ${propsClassName}` : "");
+  }
+  const className =
+    "text-sm font-semibold rounded-md px-3 py-2 shadow-sm focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 transition-all duration-150" +
+    getVariantClassNames(variant) +
+    (propsClassName ? ` ${propsClassName}` : "");
   // END TODO
 
-  return(
+  return (
     <Link {...rest}>
       <button className={className}>
-        {icon && <FontAwesomeIcon className="mr-2" icon={icon}/>}
+        {icon && <FontAwesomeIcon className="mr-2" icon={icon} />}
         {children}
       </button>
     </Link>
   );
- }
\ No newline at end of file
+};
diff --git a/app/modules/TopBar/TopBar.tsx b/app/modules/TopBar/TopBar.tsx
index 810f459186..bdccc86076 100644
--- a/app/modules/TopBar/TopBar.tsx
+++ b/app/modules/TopBar/TopBar.tsx
@@ -15,7 +15,7 @@ export const TopBar = () => {
       border-b border-ui-panel-border bg-ui-panel"
     >
       <nav
-        className="mx-auto flex max-w-full items-center justify-between px-4 py-3"
+        className="mx-auto flex max-w-full items-center justify-between p-4"
         aria-label="Global"
       >
         <div className="grid w-full grid-cols-3 gap-2">
diff --git a/app/pages/PageEnigma/PageEnigma.tsx b/app/pages/PageEnigma/PageEnigma.tsx
index 9248f1c8d5..108c9b1867 100644
--- a/app/pages/PageEnigma/PageEnigma.tsx
+++ b/app/pages/PageEnigma/PageEnigma.tsx
@@ -1,11 +1,8 @@
 import { useCallback, useEffect, useRef, useState } from "react";
 
-import {
-  faChevronLeft,
-  faWandSparkles,
-} from "@fortawesome/pro-solid-svg-icons";
+import { faWandSparkles } from "@fortawesome/pro-solid-svg-icons";
 
-import { Button, ButtonLink } from "~/components";
+import { Button } from "~/components";
 import { ButtonDialogue } from "~/modules/ButtonDialogue";
 import { TopBarHelmet } from "~/modules/TopBarHelmet/TopBarHelmet";
 // import { SidePanel } from "~/modules/SidePanel";
@@ -55,6 +52,7 @@ export const PageEnigma = () => {
     }
   }, []);
 
+  //for updating timeline/engine div height (for resizing)
   useEffect(() => {
     const observer = new ResizeObserver((entries) => {
       for (let entry of entries) {
@@ -75,7 +73,7 @@ export const PageEnigma = () => {
         <Button icon={faWandSparkles}>Generate Movie</Button>
       </TopBarHelmet>
 
-      <div style={{ height: "calc(100vh - 64px)" }}>
+      <div style={{ height: "calc(100vh - 68px)" }}>
         {/* Engine section/side panel */}
         <div
           className="flex"
diff --git a/app/styles/base.css b/app/styles/base.css
index f4da4f836b..f1a3d55366 100644
--- a/app/styles/base.css
+++ b/app/styles/base.css
@@ -3,6 +3,6 @@ body {
 }
 
 .topbar-spacer {
-  height: 64px;
+  height: 68px;
   width: 100vw;
 }