Skip to content

Commit 4dc66be

Browse files
authored
Merge pull request #319 from dd3tech/fix/file-item-export
Fix: file item export
2 parents 6f00552 + caab233 commit 4dc66be

File tree

6 files changed

+29
-29
lines changed

6 files changed

+29
-29
lines changed

exports.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@
155155
"./FileItem": {
156156
"require": "./dist/cjs/components/Form/File/FileItem.js",
157157
"module": "./dist/esm/components/Form/File/FileItem.js",
158-
"default": "./dist/esm/components/Form/File/FileList.js"
158+
"default": "./dist/esm/components/Form/File/FileItem.js"
159159
},
160160
"./FileImageItem": {
161161
"require": "./dist/cjs/components/Form/File/FileImageItem.js",

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,7 +291,7 @@
291291
"./FileItem": {
292292
"require": "./dist/cjs/components/Form/File/FileItem.js",
293293
"module": "./dist/esm/components/Form/File/FileItem.js",
294-
"default": "./dist/esm/components/Form/File/FileList.js"
294+
"default": "./dist/esm/components/Form/File/FileItem.js"
295295
},
296296
"./FileImageItem": {
297297
"require": "./dist/cjs/components/Form/File/FileImageItem.js",

src/components/Form/File/FileItem.test.tsx

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,6 @@ describe('<FileItem>', () => {
1414
expect(getByRole('file-item')).toBeDefined()
1515
})
1616

17-
it('displays file size when provided', () => {
18-
const { getByRole } = render(<FileItem {...defaultProps} />)
19-
20-
expect(getByRole('file-size')).toBeDefined()
21-
})
22-
2317
it('renders children components', () => {
2418
const { getByTestId } = render(
2519
<FileItem {...defaultProps}>

src/components/Form/File/FileItem.tsx

Lines changed: 26 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import { CSSProperties, ReactNode } from 'react'
22
import { composeClasses } from 'lib/classes'
33
import { Flex } from 'components/Layout'
4-
import ToolTipHover from 'components/ToolTipHover'
54
import Text from 'components/Typography'
5+
import Tooltip from 'components/Tooltip'
66
import FileItemButton from './FileItemButton'
77
import FileItemDropdown from './FileItemDropdown'
88

@@ -54,22 +54,32 @@ export const FileItem = ({
5454
}}
5555
>
5656
<Flex alignItems="center" gap="2" className="mr-auto overflow-hidden">
57-
<ToolTipHover
58-
variantPopup="dark"
59-
complementPosition={{ top: -15, left: 0 }}
60-
element={
61-
<Flex alignItems="center" justifyContent="start">
62-
<div className="truncate">
63-
<Text role="file-title" size="xs">
64-
{name}
65-
</Text>
66-
</div>
67-
{fileSize && (
68-
<Text role="file-size" className="mt-0.5 ml-0.5" size="xs">
69-
{fileSize} KB
57+
<Tooltip
58+
variant="primary"
59+
content={
60+
<>
61+
<Flex alignItems="center" justifyContent="start">
62+
<div className="truncate">
63+
<Text role="file-title" size="xs">
64+
{name}
65+
</Text>
66+
</div>
67+
{fileSize && (
68+
<Text role="file-size" className="mt-0.5 ml-0.5" size="xs">
69+
{fileSize} KB
70+
</Text>
71+
)}
72+
</Flex>
73+
{type && (
74+
<Text
75+
role="file-type"
76+
variant="small"
77+
className="block text-left"
78+
>
79+
{type}
7080
</Text>
7181
)}
72-
</Flex>
82+
</>
7383
}
7484
>
7585
<Text
@@ -79,10 +89,7 @@ export const FileItem = ({
7989
>
8090
{name}
8191
</Text>
82-
<Text role="file-type" variant="small" className="block text-left">
83-
{type}
84-
</Text>
85-
</ToolTipHover>
92+
</Tooltip>
8693
</Flex>
8794
<div
8895
role="file-actions"

src/components/Form/File/FileItemButton.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { ReactNode } from 'react'
22
import { composeClasses } from 'lib/classes'
33
import { Button, IButtonProps } from 'components/Buttons'
44

5-
export interface FileItemButtonProps {
5+
export interface FileItemButtonProps extends Omit<IButtonProps, 'children'> {
66
/**
77
* Content to display inside the button
88
*/

src/stories/InputFile.stories.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ type Story = StoryObj<typeof InputFile>
1212

1313
export const Default: Story = {
1414
args: {
15-
hintText: 'Subir en formato XML - PDF',
1615
progressIndicator: 0,
1716
dragMessage: 'Suelta aquí',
1817
id: 'upload',

0 commit comments

Comments
 (0)