Skip to content

Commit 21b40bb

Browse files
fix(footer): update tooltip implementation to match blueprint-web types
Co-Authored-By: gregorywong@box.com <gregorywong@box.com>
1 parent fafbde1 commit 21b40bb

File tree

1 file changed

+19
-12
lines changed

1 file changed

+19
-12
lines changed

src/elements/content-picker/Footer.tsx

Lines changed: 19 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import * as React from 'react';
22
import { injectIntl, type IntlShape } from 'react-intl';
3-
import { Button, Toolbar } from '@box/blueprint-web';
3+
import { Button, Toolbar, Tooltip } from '@box/blueprint-web';
44
import type { Collection, BoxItem } from '../../common/types/core';
55

66
import messages from '../common/messages';
@@ -77,18 +77,25 @@ const Footer = ({
7777
})
7878
) : (
7979
<Toolbar.Root className="bcp-footer-actions">
80-
<Button onClick={onCancel} variant="secondary">
81-
{cancelButtonLabel || cancelMessage}
82-
</Button>
83-
<Button
84-
disabled={isChooseButtonDisabled}
85-
onClick={onChoose}
86-
variant="primary"
87-
data-testid="choose-button"
88-
data-resin-target="choose"
80+
<Tooltip content={cancelButtonLabel || cancelMessage}>
81+
<Button onClick={onCancel} variant="secondary">
82+
{cancelButtonLabel || cancelMessage}
83+
</Button>
84+
</Tooltip>
85+
<Tooltip
86+
content={chooseButtonLabel || chooseMessage}
87+
open={isChooseButtonDisabled ? false : undefined}
8988
>
90-
{chooseButtonLabel || chooseMessage}
91-
</Button>
89+
<Button
90+
disabled={isChooseButtonDisabled}
91+
onClick={onChoose}
92+
variant="primary"
93+
data-testid="choose-button"
94+
data-resin-target="choose"
95+
>
96+
{chooseButtonLabel || chooseMessage}
97+
</Button>
98+
</Tooltip>
9299
</Toolbar.Root>
93100
)}
94101
</div>

0 commit comments

Comments
 (0)