fix(Ellipsis): the whiteSpace style of the calc container should be…

… the same as Ellipsis (#6342)
ant-design · Aug 29, 2023 · bb991b8 · bb991b8
1 parent 681671d
commit bb991b8
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/src/components/ellipsis/ellipsis.tsx b/src/components/ellipsis/ellipsis.tsx
@@ -74,7 +74,6 @@ export const Ellipsis: FC<EllipsisProps> = p => {
     container.style.minHeight = 'auto'
     container.style.maxHeight = 'auto'
     container.style.textOverflow = 'clip'
-    container.style.whiteSpace = 'normal'
     container.style.webkitLineClamp = 'unset'
     container.style.display = 'block'
 

diff --git a/src/components/ellipsis/tests/ellipsis.test.tsx b/src/components/ellipsis/tests/ellipsis.test.tsx
@@ -151,4 +151,22 @@ describe('Ellipsis', () => {
     fireEvent.click(getByText('expand'))
     expect(getByText('collapse')).toBeInTheDocument()
   })
+
+  test('the `whiteSpace` style of the calc container should be the same as Ellipsis', () => {
+    let container: Element | null = null
+    Object.defineProperty(HTMLElement.prototype, 'removeChild', {
+      value: (child: Element) => {
+        container = child
+      },
+    })
+    render(
+      <Ellipsis
+        style={{
+          whiteSpace: 'pre-wrap',
+        }}
+        content={content}
+      />
+    )
+    expect(container).toHaveStyle('white-space: pre-wrap')
+  })
 })