From f11686257c97c3cbcd1a48a3b3d51a7d8a739be6 Mon Sep 17 00:00:00 2001
From: Ashish Gupta <ashish@getcollate.io>
Date: Tue, 13 May 2025 15:16:33 +0530
Subject: [PATCH] fix lineage column icon and entity summary panel owner label
 not visible (#21160)

---
 .../Entity/EntityLineage/custom-node.less     |  6 +-
 .../CommonEntitySummaryInfo.tsx               | 84 ++++++++-----------
 2 files changed, 34 insertions(+), 56 deletions(-)

diff --git a/openmetadata-ui/src/main/resources/ui/src/components/Entity/EntityLineage/custom-node.less b/openmetadata-ui/src/main/resources/ui/src/components/Entity/EntityLineage/custom-node.less
index 40e96be1207..374908dc9c8 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/Entity/EntityLineage/custom-node.less
+++ b/openmetadata-ui/src/main/resources/ui/src/components/Entity/EntityLineage/custom-node.less
@@ -86,16 +86,12 @@
       border-radius: 0;
 
       .ant-collapse-header {
-        padding: 0;
+        padding: 4px;
         font-size: 12px;
 
         .custom-node-column-container {
           background-color: @lineage-collapse-header;
         }
-
-        .lineage-column-node-handle {
-          background-color: @lineage-collapse-header;
-        }
       }
 
       .ant-collapse-content-box {
diff --git a/openmetadata-ui/src/main/resources/ui/src/components/Explore/EntitySummaryPanel/CommonEntitySummaryInfo/CommonEntitySummaryInfo.tsx b/openmetadata-ui/src/main/resources/ui/src/components/Explore/EntitySummaryPanel/CommonEntitySummaryInfo/CommonEntitySummaryInfo.tsx
index f180d3357ea..8581514d0d2 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/Explore/EntitySummaryPanel/CommonEntitySummaryInfo/CommonEntitySummaryInfo.tsx
+++ b/openmetadata-ui/src/main/resources/ui/src/components/Explore/EntitySummaryPanel/CommonEntitySummaryInfo/CommonEntitySummaryInfo.tsx
@@ -31,61 +31,43 @@ function CommonEntitySummaryInfo({
   return (
     <Row className="text-sm" gutter={[0, 4]}>
       {entityInfo.map((info) => {
-        const isOwner = info.name === t('label.owner-plural');
         const isDomain = isDomainVisible && info.name === t('label.domain');
 
         return info.visible?.includes(componentType) || isDomain ? (
           <Col key={info.name} span={24}>
-            <Row
-              className={classNames('', {
-                'p-b-md': isOwner,
-              })}
-              gutter={[16, 32]}>
-              {!isOwner ? (
-                <>
-                  <Col span={8}>
-                    <Typography.Text
-                      className="summary-item-key font-semibold"
-                      data-testid={`${info.name}-label`}>
-                      {info.name}
-                    </Typography.Text>
-                  </Col>
-                  <Col span={16}>
-                    {info.isLink ? (
-                      <Link
-                        component={Typography.Link}
-                        data-testid={`${info.name}-value`}
-                        target={info.isExternal ? '_blank' : '_self'}
-                        to={info.linkProps ?? { pathname: info.url }}>
-                        {info.value}
-                        {info.isExternal ? (
-                          <Icon
-                            className="m-l-xs"
-                            component={IconExternalLink}
-                            data-testid="external-link-icon"
-                            style={ICON_DIMENSION}
-                          />
-                        ) : null}
-                      </Link>
-                    ) : (
-                      <Typography.Text
-                        className={classNames(
-                          'summary-item-value text-grey-muted',
-                          {
-                            'text-grey-body': !isOwner,
-                          }
-                        )}
-                        data-testid={`${info.name}-value`}>
-                        {info.value}
-                      </Typography.Text>
-                    )}
-                  </Col>
-                </>
-              ) : (
-                <Col data-testid={`${info.name}-value`} span={24}>
-                  {info.value}
-                </Col>
-              )}
+            <Row gutter={[16, 32]}>
+              <Col span={8}>
+                <Typography.Text
+                  className="summary-item-key font-semibold"
+                  data-testid={`${info.name}-label`}>
+                  {info.name}
+                </Typography.Text>
+              </Col>
+              <Col span={16}>
+                {info.isLink ? (
+                  <Link
+                    component={Typography.Link}
+                    data-testid={`${info.name}-value`}
+                    target={info.isExternal ? '_blank' : '_self'}
+                    to={info.linkProps ?? { pathname: info.url }}>
+                    {info.value}
+                    {info.isExternal ? (
+                      <Icon
+                        className="m-l-xs"
+                        component={IconExternalLink}
+                        data-testid="external-link-icon"
+                        style={ICON_DIMENSION}
+                      />
+                    ) : null}
+                  </Link>
+                ) : (
+                  <Typography.Text
+                    className={classNames('summary-item-value text-grey-body')}
+                    data-testid={`${info.name}-value`}>
+                    {info.value}
+                  </Typography.Text>
+                )}
+              </Col>
             </Row>
           </Col>
         ) : null;