Remove else if on position calculation (#5691)

As title fix
twentyhq · May 31, 2024 · 1766eb5 · 1766eb5
1 parent d9f7b18
commit 1766eb5
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 15 deletions.
diff --git a/.../twenty-front/src/modules/activities/timelineActivities/components/TimelineActivities.tsx b/.../twenty-front/src/modules/activities/timelineActivities/components/TimelineActivities.tsx
@@ -25,14 +25,10 @@ const StyledMainContainer = styled.div`
   height: 100%;
 
   justify-content: center;
-  padding: ${({ theme }) =>
-    theme.spacing(6) +
-    ' ' +
-    theme.spacing(6) +
-    ' ' +
-    theme.spacing(16) +
-    ' ' +
-    theme.spacing(6)};
+  padding-top: ${({ theme }) => theme.spacing(6)};
+  padding-right: ${({ theme }) => theme.spacing(6)};
+  padding-bottom: ${({ theme }) => theme.spacing(16)};
+  padding-left: ${({ theme }) => theme.spacing(6)};
   gap: ${({ theme }) => theme.spacing(4)};
 `;
 

diff --git a/...ty-front/src/modules/object-record/record-board/utils/get-dragged-record-position.util.ts b/...ty-front/src/modules/object-record/record-board/utils/get-dragged-record-position.util.ts
@@ -6,11 +6,15 @@ export const getDraggedRecordPosition = (
 ): number => {
   if (isDefined(recordAfterPosition) && isDefined(recordBeforePosition)) {
     return (recordBeforePosition + recordAfterPosition) / 2;
-  } else if (isDefined(recordAfterPosition)) {
+  }
+
+  if (isDefined(recordAfterPosition)) {
     return recordAfterPosition - 1;
-  } else if (isDefined(recordBeforePosition)) {
+  }
+
+  if (isDefined(recordBeforePosition)) {
     return recordBeforePosition + 1;
-  } else {
-    return 1;
   }
+
+  return 1;
 };
diff --git a/packages/twenty-front/src/modules/ui/layout/show-page/components/ShowPageRightContainer.tsx b/packages/twenty-front/src/modules/ui/layout/show-page/components/ShowPageRightContainer.tsx
@@ -80,7 +80,7 @@ export const ShowPageRightContainer = ({
   const shouldDisplayLogTab = useIsFeatureEnabled('IS_EVENT_OBJECT_ENABLED');
   const shouldDisplayEmailsTab = emails && isCompanyOrPerson;
 
-  const TAB_LIST = [
+  const tabs = [
     {
       id: 'timeline',
       title: 'Timeline',
@@ -131,7 +131,7 @@ export const ShowPageRightContainer = ({
       case 'calendar':
         return <Calendar targetableObject={targetableObject} />;
       default:
-        return null;
+        return <></>;
     }
   };
 
@@ -141,7 +141,7 @@ export const ShowPageRightContainer = ({
         <TabList
           loading={loading}
           tabListId={TAB_LIST_COMPONENT_ID}
-          tabs={TAB_LIST}
+          tabs={tabs}
         />
       </StyledTabListContainer>
       {renderActiveTabContent()}