From b38000c343a6d1bfc1567b4f337c1e3f01b31abd Mon Sep 17 00:00:00 2001
From: Siarhei Karol <skarol@ebsco.com>
Date: Wed, 25 Dec 2024 15:47:26 +0300
Subject: [PATCH] Edit headers in full entity display screens

---
 src/common/constants/uiControls.constants.ts  |  1 +
 src/components/FullDisplay/PreviewContent.tsx |  2 --
 src/components/Preview/Fields.tsx             |  3 ++-
 src/components/Preview/Labels.tsx             |  1 -
 src/components/Preview/Preview.scss           | 25 ++-----------------
 .../__tests__/components/FullDisplay.test.tsx |  3 +--
 6 files changed, 6 insertions(+), 29 deletions(-)

diff --git a/src/common/constants/uiControls.constants.ts b/src/common/constants/uiControls.constants.ts
index 43fbd639..cc887555 100644
--- a/src/common/constants/uiControls.constants.ts
+++ b/src/common/constants/uiControls.constants.ts
@@ -34,6 +34,7 @@ export const UI_DROPDOWNS_LIST = [AdvancedFieldType.dropdown, AdvancedFieldType.
 
 export const NOT_PREVIEWABLE_TYPES = [
   AdvancedFieldType.profile,
+  AdvancedFieldType.block,
   AdvancedFieldType.hidden,
   AdvancedFieldType.dropdownOption,
   AdvancedFieldType.complex,
diff --git a/src/components/FullDisplay/PreviewContent.tsx b/src/components/FullDisplay/PreviewContent.tsx
index b3cad58d..0c36fa57 100644
--- a/src/components/FullDisplay/PreviewContent.tsx
+++ b/src/components/FullDisplay/PreviewContent.tsx
@@ -15,7 +15,6 @@ export const PreviewContent = () => {
 
   return previewContent.map(({ id, base, userValues, initKey, title, entities }) => {
     const resourceType = entities?.map(e => RESOURCE_TEMPLATE_IDS[e])?.[0];
-    const resourceTypeWithFallback = resourceType ?? '-';
     const handleButtonClick = () => setPreviewContent(previewContent.filter(entry => entry.id !== id));
 
     return (
@@ -31,7 +30,6 @@ export const PreviewContent = () => {
           </Button>
           <div className="info">
             <span className="title">{title}</span>
-            <span className="type">{resourceTypeWithFallback}</span>
           </div>
           <Button
             data-testid="preview-fetch"
diff --git a/src/components/Preview/Fields.tsx b/src/components/Preview/Fields.tsx
index 53d65bf2..2304fad9 100644
--- a/src/components/Preview/Fields.tsx
+++ b/src/components/Preview/Fields.tsx
@@ -79,8 +79,9 @@ export const Fields = ({
   const entry = base.get(uuid);
   const isOnBranchWithUserValue = paths.includes(uuid);
   const isEntity = level === ENTITY_LEVEL;
+  const hasEmptyChildren = entry?.children?.every(id => !schema.get(id));
 
-  if (!entry) return null;
+  if (!entry || hasEmptyChildren) return null;
 
   const { displayName = '', children, type, bfid = '', path, htmlId } = entry;
   const isDependentDropdownOption =
diff --git a/src/components/Preview/Labels.tsx b/src/components/Preview/Labels.tsx
index d37ded2a..3e17e6fd 100644
--- a/src/components/Preview/Labels.tsx
+++ b/src/components/Preview/Labels.tsx
@@ -22,7 +22,6 @@ export const Labels: FC<LabelProps> = ({
   return (
     <strong
       className={classNames({
-        'entity-heading': isEntity,
         'sub-heading': isBlock,
         'value-heading': !isGroupable,
       })}
diff --git a/src/components/Preview/Preview.scss b/src/components/Preview/Preview.scss
index a9dbc66a..9480d0b0 100644
--- a/src/components/Preview/Preview.scss
+++ b/src/components/Preview/Preview.scss
@@ -9,7 +9,7 @@
   .preview-entity {
     max-width: 560px;
     width: 100%;
-    
+
     &:not(:last-child) {
       border-right: 1px solid rgba(0, 0, 0, 0.1);
       padding-right: 0.938rem;
@@ -42,27 +42,6 @@
     }
   }
 
-  .entity-heading {
-    display: flex;
-    align-items: center;
-    gap: 0.313rem;
-    font-size: 1.138rem;
-    padding-bottom: 0.5rem;
-    
-    .toggle-entity-edit {
-      margin-left: auto;
-    }
-
-    > svg {
-      height: 1.25rem;
-      width: 1.25rem;
-
-      & * {
-        fill: #00000099;
-      }
-    }
-  }
-
   .value-heading {
     padding-bottom: 0.219rem;
   }
@@ -109,7 +88,7 @@
       }
 
       span {
-        color: rgba(0, 0, 0, 0.6)
+        color: rgba(0, 0, 0, 0.6);
       }
     }
   }
diff --git a/src/test/__tests__/components/FullDisplay.test.tsx b/src/test/__tests__/components/FullDisplay.test.tsx
index fa64c8b4..3ebb7781 100644
--- a/src/test/__tests__/components/FullDisplay.test.tsx
+++ b/src/test/__tests__/components/FullDisplay.test.tsx
@@ -58,11 +58,10 @@ describe('FullDisplay', () => {
     );
   });
 
-  const { getByTestId, getAllByTestId, getByText } = screen;
+  const { getByTestId, getAllByTestId } = screen;
 
   test('contains preview container and header', () => {
     expect(getByTestId('preview-contents-container')).toBeInTheDocument();
-    expect(getByText('Work')).toBeInTheDocument();
   });
 
   test('removes a preview content entry on close button click', () => {