fix issue (#7152 : Improve relation empty states on record page) (#7157)

- "No xxx" removed for empty relations
- All(0) removed

---------

Co-authored-by: Raphaël Bosi <71827178+bosiraphael@users.noreply.github.com>

packages/twenty-front/src/modules/object-record/record-show/record-detail-section/components/RecordDetailRelationRecordsListEmptyState.tsx

@@ -1,35 +0,0 @@
-import { useTheme } from '@emotion/react';
-import styled from '@emotion/styled';
-import { useIcons } from 'twenty-ui';
-
-import { ObjectMetadataItem } from '@/object-metadata/types/ObjectMetadataItem';
-
-type RecordDetailRelationRecordsListEmptyStateProps = {
-  relationObjectMetadataItem: ObjectMetadataItem;
-};
-
-const StyledRelationRecordsListEmptyState = styled.div`
-  color: ${({ theme }) => theme.font.color.light};
-  align-items: center;
-  justify-content: center;
-  gap: ${({ theme }) => theme.spacing(2)};
-  display: flex;
-  height: ${({ theme }) => theme.spacing(10)};
-  text-transform: capitalize;
-`;
-
-export const RecordDetailRelationRecordsListEmptyState = ({
-  relationObjectMetadataItem,
-}: RecordDetailRelationRecordsListEmptyStateProps) => {
-  const theme = useTheme();
-
-  const { getIcon } = useIcons();
-  const Icon = getIcon(relationObjectMetadataItem.icon);
-
-  return (
-    <StyledRelationRecordsListEmptyState>
-      <Icon size={theme.icon.size.sm} />
-      <div>No {relationObjectMetadataItem.labelSingular}</div>
-    </StyledRelationRecordsListEmptyState>
-  );
-};

packages/twenty-front/src/modules/object-record/record-show/record-detail-section/components/RecordDetailRelationSection.tsx

@@ -13,8 +13,6 @@ import { RelationFromManyFieldInputMultiRecordsEffect } from '@/object-record/re
 import { useUpdateRelationFromManyFieldInput } from '@/object-record/record-field/meta-types/input/hooks/useUpdateRelationFromManyFieldInput';
 import { FieldRelationMetadata } from '@/object-record/record-field/types/FieldMetadata';
 import { RecordDetailRelationRecordsList } from '@/object-record/record-show/record-detail-section/components/RecordDetailRelationRecordsList';
-import { RecordDetailRelationRecordsListEmptyState } from '@/object-record/record-show/record-detail-section/components/RecordDetailRelationRecordsListEmptyState';
-import { RecordDetailRelationSectionSkeletonLoader } from '@/object-record/record-show/record-detail-section/components/RecordDetailRelationSectionSkeletonLoader';
 import { RecordDetailSection } from '@/object-record/record-show/record-detail-section/components/RecordDetailSection';
 import { RecordDetailSectionHeader } from '@/object-record/record-show/record-detail-section/components/RecordDetailSectionHeader';
 import { recordStoreFamilyState } from '@/object-record/record-store/states/recordStoreFamilyState';
@@ -133,20 +131,10 @@ export const RecordDetailRelationSection = ({
   }?${qs.stringify(filterQueryParams)}`;
 
   const showContent = () => {
-    if (loading) {
-      return (
-        <RecordDetailRelationSectionSkeletonLoader
-          numSkeletons={fieldName === 'people' ? 2 : 1}
-        />
-      );
-    }
-
-    return relationRecords.length ? (
-      <RecordDetailRelationRecordsList relationRecords={relationRecords} />
-    ) : (
-      <RecordDetailRelationRecordsListEmptyState
-        relationObjectMetadataItem={relationObjectMetadataItem}
-      />
+    return (
+      relationRecords.length > 0 && (
+        <RecordDetailRelationRecordsList relationRecords={relationRecords} />
+      )
     );
   };
 
@@ -158,6 +146,8 @@ export const RecordDetailRelationSection = ({
       recordId,
     });
 
+  if (loading) return null;
+
   return (
     <RecordDetailSection>
       <RecordDetailSectionHeader
@@ -166,11 +156,15 @@ export const RecordDetailRelationSection = ({
           isToManyObjects
             ? {
                 to: filterLinkHref,
-                label: `All (${relationRecords.length})`,
+                label:
+                  relationRecords.length > 0
+                    ? `All (${relationRecords.length})`
+                    : '',
               }
             : undefined
         }
         hideRightAdornmentOnMouseLeave={!isDropdownOpen && !isMobile}
+        areRecordsAvailable={relationRecords.length > 0}
         rightAdornment={
           <DropdownScope dropdownScopeId={dropdownId}>
             <StyledAddDropdown

packages/twenty-front/src/modules/object-record/record-show/record-detail-section/components/RecordDetailRelationSectionSkeletonLoader.tsx

@@ -1,31 +0,0 @@
-import Skeleton, { SkeletonTheme } from 'react-loading-skeleton';
-import { useTheme } from '@emotion/react';
-import styled from '@emotion/styled';
-
-const StyledSkeletonDiv = styled.div`
-  display: flex;
-  flex-direction: column;
-  gap: ${({ theme }) => theme.spacing(4)};
-  height: 40px;
-`;
-
-export const RecordDetailRelationSectionSkeletonLoader = ({
-  numSkeletons = 1,
-}: {
-  numSkeletons?: number;
-}) => {
-  const theme = useTheme();
-  return (
-    <SkeletonTheme
-      baseColor={theme.background.tertiary}
-      highlightColor={theme.background.transparent.lighter}
-      borderRadius={4}
-    >
-      <StyledSkeletonDiv>
-        {Array.from({ length: numSkeletons }).map((_, index) => (
-          <Skeleton key={index} width={129} height={16} />
-        ))}
-      </StyledSkeletonDiv>
-    </SkeletonTheme>
-  );
-};

packages/twenty-front/src/modules/object-record/record-show/record-detail-section/components/RecordDetailSectionHeader.tsx

@@ -1,12 +1,16 @@
+import styled from '@emotion/styled';
 import { useState } from 'react';
 import { Link } from 'react-router-dom';
-import styled from '@emotion/styled';
 
-const StyledHeader = styled.header<{ isDropdownOpen?: boolean }>`
+const StyledHeader = styled.header<{
+  isDropdownOpen?: boolean;
+  areRecordsAvailable?: boolean;
+}>`
   align-items: center;
   display: flex;
   height: 24px;
-  margin-bottom: ${({ theme }) => theme.spacing(2)};
+  margin-bottom: ${({ theme, areRecordsAvailable }) =>
+    areRecordsAvailable && theme.spacing(2)};
 `;
 
 const StyledTitle = styled.div`
@@ -34,6 +38,7 @@ type RecordDetailSectionHeaderProps = {
   link?: { to: string; label: string };
   rightAdornment?: React.ReactNode;
   hideRightAdornmentOnMouseLeave?: boolean;
+  areRecordsAvailable?: boolean;
 };
 
 export const RecordDetailSectionHeader = ({
@@ -41,11 +46,13 @@ export const RecordDetailSectionHeader = ({
   link,
   rightAdornment,
   hideRightAdornmentOnMouseLeave = true,
+  areRecordsAvailable = false,
 }: RecordDetailSectionHeaderProps) => {
   const [isHovered, setIsHovered] = useState(false);
 
   return (
     <StyledHeader
+      areRecordsAvailable={areRecordsAvailable}
       onMouseEnter={() => setIsHovered(true)}
       onMouseLeave={() => setIsHovered(false)}
     >
@@ -53,7 +60,9 @@ export const RecordDetailSectionHeader = ({
         <StyledTitleLabel>{title}</StyledTitleLabel>
         {link && <StyledLink to={link.to}>{link.label}</StyledLink>}
       </StyledTitle>
-      {hideRightAdornmentOnMouseLeave && !isHovered ? null : rightAdornment}
+      {hideRightAdornmentOnMouseLeave && !isHovered && areRecordsAvailable
+        ? null
+        : rightAdornment}
     </StyledHeader>
   );
 };