From 9c9bc0d76137708cf067026b10131b8ed9f69898 Mon Sep 17 00:00:00 2001
From: e-for-eshaan <eshaanyadav24@gmail.com>
Date: Thu, 16 May 2024 17:36:18 +0530
Subject: [PATCH 1/5] fixes spacing for team-cards for a single team

---
 web-server/src/components/TeamsList.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web-server/src/components/TeamsList.tsx b/web-server/src/components/TeamsList.tsx
index 12014dc7c..a3e3868a3 100644
--- a/web-server/src/components/TeamsList.tsx
+++ b/web-server/src/components/TeamsList.tsx
@@ -76,7 +76,7 @@ export const TeamsList = () => {
           gap={4}
           grid
           gridTemplateColumns={'1fr 1fr'}
-          maxWidth={'900px'}
+          width={'900px'}
           relative
           sx={{
             filter: isLoadingTeams ? 'blur(2px)' : 'none',

From 3c3a18e0896fbd732b399b34195aee40b30bedf9 Mon Sep 17 00:00:00 2001
From: e-for-eshaan <eshaanyadav24@gmail.com>
Date: Thu, 16 May 2024 17:47:16 +0530
Subject: [PATCH 2/5] fixes spacings, and dims the integration heading on
 DORA-CTA

---
 web-server/pages/integrations.tsx | 22 +++++++++++++++++-----
 1 file changed, 17 insertions(+), 5 deletions(-)

diff --git a/web-server/pages/integrations.tsx b/web-server/pages/integrations.tsx
index 26cbad1b4..d449e5b45 100644
--- a/web-server/pages/integrations.tsx
+++ b/web-server/pages/integrations.tsx
@@ -1,6 +1,6 @@
 import { Add } from '@mui/icons-material';
 import { Button, Divider } from '@mui/material';
-import { useEffect } from 'react';
+import { useEffect, useMemo } from 'react';
 import { Authenticated } from 'src/components/Authenticated';
 
 import { FlexBox } from '@/components/FlexBox';
@@ -80,13 +80,25 @@ const Content = () => {
     }
   }, [dispatch, isGithubIntegrated, loadedTeams.true, orgId, teamCount]);
 
+  const showDoraCTA = useMemo(
+    () => Boolean(teamCount) && Boolean(hasCodeProviderLinked),
+    [hasCodeProviderLinked, teamCount]
+  );
+
   return (
     <FlexBox col gap2>
-      <FlexBox justifyBetween>
-        <Line white fontSize={'24px'}>
+      <FlexBox justifyBetween height={'52px'} alignCenter>
+        <Line
+          white
+          fontSize={'24px'}
+          sx={{
+            opacity: showDoraCTA || showCreationCTA ? 0.4 : 1,
+            transition: 'all 0.2s ease'
+          }}
+        >
           Integrate your Github to fetch DORA for your team
         </Line>
-        {Boolean(teamCount) && Boolean(hasCodeProviderLinked) && (
+        {showDoraCTA && (
           <Button href={ROUTES.DORA_METRICS.PATH} variant="contained">
             <FlexBox centered fullWidth p={2 / 3}>
               Continue to Dora {'->'}
@@ -104,7 +116,7 @@ const Content = () => {
           <Line fontSize={'24px'} semibold white>
             Create team structure to see DORA
           </Line>
-          <Line fontSize={'16px'} white mt="14px">
+          <Line fontSize={'16px'} mt="9px">
             Just add team's name, add repos and you're good to go.
           </Line>
           <Button

From cf5ffd221645aca88994d8dc3aabaf29b470962e Mon Sep 17 00:00:00 2001
From: e-for-eshaan <eshaanyadav24@gmail.com>
Date: Thu, 16 May 2024 18:21:50 +0530
Subject: [PATCH 3/5] removes the animation from the image, fixes CTA for
 gh-PAT

---
 .../Dashboards/useIntegrationHandlers.tsx     | 97 ++++++-------------
 1 file changed, 28 insertions(+), 69 deletions(-)

diff --git a/web-server/src/content/Dashboards/useIntegrationHandlers.tsx b/web-server/src/content/Dashboards/useIntegrationHandlers.tsx
index 19edd46fb..4874ccc2b 100644
--- a/web-server/src/content/Dashboards/useIntegrationHandlers.tsx
+++ b/web-server/src/content/Dashboards/useIntegrationHandlers.tsx
@@ -1,8 +1,8 @@
 import { LoadingButton } from '@mui/lab';
-import { Divider, Link, SxProps, TextField, alpha } from '@mui/material';
+import { Divider, Link, TextField, alpha } from '@mui/material';
 import Image from 'next/image';
 import { useSnackbar } from 'notistack';
-import { FC, useCallback, useEffect, useMemo } from 'react';
+import { FC, useCallback, useMemo } from 'react';
 
 import { FlexBox } from '@/components/FlexBox';
 import { Line } from '@/components/Text';
@@ -132,7 +132,7 @@ const ConfigureGithubModalBody: FC<{
 
   return (
     <FlexBox gap2>
-      <FlexBox gap={2} minWidth={'400px'} maxHeight={'255px'} col>
+      <FlexBox gap={2} minWidth={'400px'} col>
         <FlexBox>Enter you Github token below.</FlexBox>
         <FlexBox fullWidth minHeight={'80px'} col>
           <TextField
@@ -156,15 +156,25 @@ const ConfigureGithubModalBody: FC<{
           <Line error tiny mt={1}>
             {showError.value}
           </Line>
-          <Line tiny mt={1}>
-            <Link
-              href="https://github.com/settings/tokens"
-              target="_blank"
-              rel="noopener noreferrer"
-            >
-              Generate new classic token
-            </Link>
-          </Line>
+          <FlexBox>
+            <Line tiny mt={1} primary sx={{ cursor: 'pointer' }}>
+              <Link
+                href="https://github.com/settings/tokens"
+                target="_blank"
+                rel="noopener noreferrer"
+              >
+                <Line
+                  underline
+                  sx={{
+                    textUnderlineOffset: '2px'
+                  }}
+                >
+                  Generate new classic token
+                </Line>
+              </Link>
+              <Line ml={'5px'}>{' ->'}</Line>
+            </Line>
+          </FlexBox>
         </FlexBox>
 
         <FlexBox justifyBetween alignCenter mt={'auto'}>
@@ -200,62 +210,14 @@ const ConfigureGithubModalBody: FC<{
 };
 
 const TokenPermissions = () => {
-  const positionArray = ['0px', '130px', '340px', '740px'];
-  const position = useEasyState(0);
-
-  const changePosition = useCallback(() => {
-    position.set((position.value + 1) % positionArray.length);
-  }, [position, positionArray.length]);
-
-  const expand = useBoolState(false);
   const imageLoaded = useBoolState(false);
 
-  // change position every second
-  useEffect(() => {
-    if (!imageLoaded.value) return;
-    if (expand.value) return depFn(position.set, 0);
-    const interval = setInterval(changePosition, 2000);
-    return () => clearInterval(interval);
-  }, [changePosition, expand.value, imageLoaded.value, position.set]);
-
-  const styles: SxProps[] = useMemo(() => {
-    const baseStyles = {
-      border: `2px solid ${alpha('rgb(256,0,0)', 0.4)}`,
-      transition: 'all 0.8s ease',
-      borderRadius: '12px',
-      opacity: expand.value ? 0 : 1,
-      width: '240px',
-      position: 'absolute',
-      maxWidth: 'calc(100% - 48px)',
-      left: '24px'
-    };
-
-    return [
-      {
-        height: '170px',
-        top: '58px'
-      },
-      {
-        height: '42px',
-        top: '98px'
-      },
-      {
-        height: '120px',
-        top: '38px'
-      },
-      {
-        height: '120px',
-        top: '66px'
-      }
-    ].map((item) => ({ ...item, ...baseStyles }));
-  }, [expand.value]);
-
   const expandedStyles = useMemo(() => {
     const baseStyles = {
       border: `2px solid ${alpha('rgb(256,0,0)', 0.4)}`,
       transition: 'all 0.8s ease',
       borderRadius: '12px',
-      opacity: !expand.value ? 0 : 1,
+      opacity: 1,
       width: '240px',
       position: 'absolute',
       maxWidth: 'calc(100% - 48px)',
@@ -282,17 +244,17 @@ const TokenPermissions = () => {
         top: '806px'
       }
     ].map((item) => ({ ...item, ...baseStyles }));
-  }, [expand.value]);
+  }, []);
 
   return (
     <FlexBox col gap1 maxWidth={'100%'} overflow={'auto'}>
       <div
-        onMouseEnter={!imageLoaded.value ? null : expand.true}
-        onMouseLeave={expand.false}
         style={{
           overflow: 'hidden',
           borderRadius: '12px',
-          height: expand.value ? '1257px' : '240px',
+          height: 'calc(100vh - 300px)',
+          maxHeight: '1257px',
+          overflowY: 'auto',
           transition: 'all 0.8s ease',
           position: 'relative',
           maxWidth: '100%',
@@ -303,7 +265,6 @@ const TokenPermissions = () => {
           onLoadingComplete={imageLoaded.true}
           style={{
             position: 'relative',
-            bottom: expand.value ? 0 : positionArray[position.value],
             transition: 'all 0.8s ease',
             opacity: !imageLoaded.value ? 0 : 1
           }}
@@ -313,8 +274,6 @@ const TokenPermissions = () => {
           alt="PAT_permissions"
         />
 
-        {imageLoaded.value && <FlexBox sx={styles[position.value]} />}
-
         {imageLoaded.value &&
           expandedStyles.map((style, index) => (
             <FlexBox key={index} sx={style} />
@@ -334,7 +293,7 @@ const TokenPermissions = () => {
         )}
       </div>
       <Line tiny secondary sx={{ opacity: imageLoaded.value ? 1 : 0 }}>
-        Hover to expand
+        Scroll to see all required permissions
       </Line>
     </FlexBox>
   );

From 676c51126c07a404831f417814bd1ba9e73d2206 Mon Sep 17 00:00:00 2001
From: e-for-eshaan <eshaanyadav24@gmail.com>
Date: Fri, 17 May 2024 07:56:00 +0530
Subject: [PATCH 4/5] fixes logo in side-menu

---
 .../ExtendedSidebarLayout/Sidebar/SidebarTopSection/index.tsx   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web-server/src/layouts/ExtendedSidebarLayout/Sidebar/SidebarTopSection/index.tsx b/web-server/src/layouts/ExtendedSidebarLayout/Sidebar/SidebarTopSection/index.tsx
index bd6c3bbb9..329858200 100644
--- a/web-server/src/layouts/ExtendedSidebarLayout/Sidebar/SidebarTopSection/index.tsx
+++ b/web-server/src/layouts/ExtendedSidebarLayout/Sidebar/SidebarTopSection/index.tsx
@@ -14,7 +14,7 @@ function SidebarTopSection() {
         position: 'relative'
       }}
     >
-      <Logo height={40} />
+      <Logo mode="long" height={'40px'} />
     </Box>
   );
 }

From cc35ad4fca3ea3d438bcef8b89f4f8fe219120da Mon Sep 17 00:00:00 2001
From: e-for-eshaan <eshaanyadav24@gmail.com>
Date: Fri, 17 May 2024 08:12:12 +0530
Subject: [PATCH 5/5] improves spacing and alignment in TeamsList component

---
 web-server/src/components/TeamsList.tsx | 35 ++++++++++++++++---------
 1 file changed, 23 insertions(+), 12 deletions(-)

diff --git a/web-server/src/components/TeamsList.tsx b/web-server/src/components/TeamsList.tsx
index a3e3868a3..fcd6914ca 100644
--- a/web-server/src/components/TeamsList.tsx
+++ b/web-server/src/components/TeamsList.tsx
@@ -29,12 +29,8 @@ import { CreateEditTeams } from './Teams/CreateTeams';
 import { Loader } from './Teams/CreateTeams';
 import { Line } from './Text';
 
-type TeamCardProps = {
-  team: Team;
-  onEdit: () => void;
-};
-
 const VISIBLE_REPOS_COUNT = 3;
+const HORIZONTAL_SPACE = 3 / 2;
 
 export const TeamsList = () => {
   const teamsArray = useSelector((state) => state.team.teams);
@@ -73,7 +69,7 @@ export const TeamsList = () => {
       ) : null}
       <FlexBox relative>
         <FlexBox
-          gap={4}
+          gap={HORIZONTAL_SPACE}
           grid
           gridTemplateColumns={'1fr 1fr'}
           width={'900px'}
@@ -115,25 +111,35 @@ const SearchFilter: FC<{
 }> = ({ searchQuery, onChange, handleShowCreateTeam, showCreate }) => {
   return (
     <FlexBox col gap={4}>
-      <FlexBox width={'900px'} gap={4}>
+      <FlexBox width={'900px'} gap={HORIZONTAL_SPACE}>
         <FlexBox flex1>
           <TextField
+            size="small"
             value={searchQuery}
             onChange={(e) => {
               onChange(e.target.value);
             }}
             fullWidth
-            placeholder="Search for teams"
+            InputProps={{
+              sx: {
+                padding: '3px'
+              }
+            }}
+            placeholder="Search"
           />
         </FlexBox>
-        <FlexBox flex1 alignCenter gap={4}>
+        <FlexBox flex1 alignCenter gap={HORIZONTAL_SPACE}>
           <FlexBox flex1>
             <Button
               onClick={handleShowCreateTeam}
-              sx={{ width: '100%' }}
               variant="outlined"
+              color="secondary"
+              sx={{
+                bgcolor: 'transparent',
+                width: '100%'
+              }}
             >
-              <FlexBox centered gap1 fullWidth p={2 / 3}>
+              <FlexBox centered gap1 fullWidth>
                 <Add fontSize="small" /> Add new team
               </FlexBox>
             </Button>
@@ -144,7 +150,7 @@ const SearchFilter: FC<{
               sx={{ width: '100%' }}
               variant="contained"
             >
-              <FlexBox centered fullWidth p={2 / 3}>
+              <FlexBox centered fullWidth>
                 Continue to Dora {'->'}
               </FlexBox>
             </Button>
@@ -161,6 +167,11 @@ const SearchFilter: FC<{
   );
 };
 
+type TeamCardProps = {
+  team: Team;
+  onEdit: () => void;
+};
+
 const TeamCard: React.FC<TeamCardProps> = ({ team, onEdit }) => {
   const { name: teamName, id: teamId } = team;
   const teamReposMap = useSelector((state) => state.team.teamReposMaps);