From a2d041e49bc889181c13499a157d537fbc74ed24 Mon Sep 17 00:00:00 2001
From: "Hugh A. Miles II" <hughmil3s@gmail.com>
Date: Tue, 17 Oct 2023 23:39:15 -0400
Subject: [PATCH] fix(tags): +n tags for listview (#25603)

---
 .../src/components/Tags/Tag.test.tsx          | 21 +++++++
 superset-frontend/src/components/Tags/Tag.tsx | 62 +++++++++----------
 .../src/components/Tags/TagsList.tsx          |  6 +-
 superset-frontend/src/types/TagType.ts        |  1 +
 4 files changed, 58 insertions(+), 32 deletions(-)

diff --git a/superset-frontend/src/components/Tags/Tag.test.tsx b/superset-frontend/src/components/Tags/Tag.test.tsx
index 0ff7b2e85a3f3..30827bafb87d2 100644
--- a/superset-frontend/src/components/Tags/Tag.test.tsx
+++ b/superset-frontend/src/components/Tags/Tag.test.tsx
@@ -18,6 +18,7 @@
  */
 import React from 'react';
 import { render } from 'spec/helpers/testing-library';
+import { screen } from '@testing-library/react';
 import TagType from 'src/types/TagType';
 import Tag from './Tag';
 
@@ -33,3 +34,23 @@ test('should render', () => {
   const { container } = render(<Tag {...mockedProps} />);
   expect(container).toBeInTheDocument();
 });
+
+test('should render shortname properly', () => {
+  const { container } = render(<Tag {...mockedProps} />);
+  expect(container).toBeInTheDocument();
+  expect(screen.getByTestId('tag')).toBeInTheDocument();
+  expect(screen.getByTestId('tag')).toHaveTextContent(mockedProps.name);
+});
+
+test('should render longname properly', () => {
+  const longNameProps = {
+    ...mockedProps,
+    name: 'very-long-tag-name-that-truncates',
+  };
+  const { container } = render(<Tag {...longNameProps} />);
+  expect(container).toBeInTheDocument();
+  expect(screen.getByTestId('tag')).toBeInTheDocument();
+  expect(screen.getByTestId('tag')).toHaveTextContent(
+    `${longNameProps.name.slice(0, 20)}...`,
+  );
+});
diff --git a/superset-frontend/src/components/Tags/Tag.tsx b/superset-frontend/src/components/Tags/Tag.tsx
index ad3df8af67f68..896535afca016 100644
--- a/superset-frontend/src/components/Tags/Tag.tsx
+++ b/superset-frontend/src/components/Tags/Tag.tsx
@@ -31,6 +31,8 @@ const StyledTag = styled(AntdTag)`
   `};
 `;
 
+const MAX_DISPLAY_CHAR = 20;
+
 const Tag = ({
   name,
   id,
@@ -38,49 +40,47 @@ const Tag = ({
   onDelete = undefined,
   editable = false,
   onClick = undefined,
+  toolTipTitle = name,
 }: TagType) => {
-  const isLongTag = useMemo(() => name.length > 20, [name]);
+  const isLongTag = useMemo(() => name.length > MAX_DISPLAY_CHAR, [name]);
+  const tagDisplay = isLongTag ? `${name.slice(0, MAX_DISPLAY_CHAR)}...` : name;
 
   const handleClose = () => (index ? onDelete?.(index) : null);
 
   const tagElem = (
     <>
       {editable ? (
-        <StyledTag
-          key={id}
-          closable={editable}
-          onClose={handleClose}
-          color="blue"
-        >
-          {isLongTag ? `${name.slice(0, 20)}...` : name}
-        </StyledTag>
+        <Tooltip title={toolTipTitle} key={toolTipTitle}>
+          <StyledTag
+            key={id}
+            closable={editable}
+            onClose={handleClose}
+            color="blue"
+          >
+            {tagDisplay}
+          </StyledTag>
+        </Tooltip>
       ) : (
-        <StyledTag role="link" key={id} onClick={onClick}>
-          {id ? (
-            <a
-              href={`/superset/all_entities/?id=${id}`}
-              target="_blank"
-              rel="noreferrer"
-            >
-              {isLongTag ? `${name.slice(0, 20)}...` : name}
-            </a>
-          ) : isLongTag ? (
-            `${name.slice(0, 20)}...`
-          ) : (
-            name
-          )}
-        </StyledTag>
+        <Tooltip title={toolTipTitle} key={toolTipTitle}>
+          <StyledTag data-test="tag" role="link" key={id} onClick={onClick}>
+            {id ? (
+              <a
+                href={`/superset/all_entities/?id=${id}`}
+                target="_blank"
+                rel="noreferrer"
+              >
+                {tagDisplay}
+              </a>
+            ) : (
+              tagDisplay
+            )}
+          </StyledTag>
+        </Tooltip>
       )}
     </>
   );
 
-  return isLongTag ? (
-    <Tooltip title={name} key={name}>
-      {tagElem}
-    </Tooltip>
-  ) : (
-    tagElem
-  );
+  return tagElem;
 };
 
 export default Tag;
diff --git a/superset-frontend/src/components/Tags/TagsList.tsx b/superset-frontend/src/components/Tags/TagsList.tsx
index 102e6d2ced31f..ea4f7e6dad4f0 100644
--- a/superset-frontend/src/components/Tags/TagsList.tsx
+++ b/superset-frontend/src/components/Tags/TagsList.tsx
@@ -83,7 +83,11 @@ const TagsList = ({
             />
           ))}
           {tags.length > tempMaxTags ? (
-            <Tag name={`+${extraTags}...`} onClick={expand} />
+            <Tag
+              name={`+${extraTags}...`}
+              onClick={expand}
+              toolTipTitle={tags.map(t => t.name).join(', ')}
+            />
           ) : null}
         </>
       ) : (
diff --git a/superset-frontend/src/types/TagType.ts b/superset-frontend/src/types/TagType.ts
index 4c1b85b724915..8f445e50dafd5 100644
--- a/superset-frontend/src/types/TagType.ts
+++ b/superset-frontend/src/types/TagType.ts
@@ -27,6 +27,7 @@ export interface TagType {
   onClick?: MouseEventHandler<HTMLSpanElement>;
   name: string;
   index?: number | undefined;
+  toolTipTitle?: string;
 }
 
 export default TagType;