fix type error on Popover close

Talend · Oct 23, 2023 · 04d9d95 · 04d9d95
1 parent 35263df
commit 04d9d95
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/packages/design-system/src/components/Popover/Popover.tsx b/packages/design-system/src/components/Popover/Popover.tsx
@@ -27,7 +27,7 @@ export type PopoverProps = {
 } & PopoverOptions;
 
 export type PopoverStateReturn = {
-	hide: () => void;
+	setOpen: () => void;
 };
 
 export function Popover({
@@ -70,7 +70,10 @@ export function Popover({
 						fill={tokens.coralColorNeutralBackground}
 					/>
 					{typeof children === 'function'
-						? children({ ...popover.getFloatingProps(), setOpen: popover.setOpen })
+						? children({
+								...popover.getFloatingProps(),
+								setOpen: popover.setOpen,
+						  })
 						: children}
 				</div>
 			</Wrapper>

diff --git a/packages/design-system/src/stories/messaging/Popover.stories.tsx b/packages/design-system/src/stories/messaging/Popover.stories.tsx
@@ -76,7 +76,7 @@ export const WithFunctionAsChildren = () => (
 			{(popover: PopoverStateReturn) => (
 				<StackVertical gap="S">
 					There is some content
-					<ButtonPrimary onClick={() => popover?.hide()}>Close Me please</ButtonPrimary>
+					<ButtonPrimary onClick={() => popover?.setOpen(false)}>Close Me please</ButtonPrimary>
 				</StackVertical>
 			)}
 			<EasyPopover />

diff --git a/packages/faceted-search/src/components/BasicSearch/BasicSearch.component.js b/packages/faceted-search/src/components/BasicSearch/BasicSearch.component.js
@@ -169,7 +169,7 @@ const BasicSearch = ({
 									initialFilterValue={initialFilterValue}
 									onClick={(...args) => {
 										onClickOverlayRow(...args);
-										popover?.hide();
+										popover?.setOpen(false);
 									}}
 									t={t}
 								/>