Skip to content

Commit 781c64f

Browse files
committed
use existing setOpen instead of new onToggle
1 parent c5d5002 commit 781c64f

File tree

2 files changed

+1
-6
lines changed

2 files changed

+1
-6
lines changed

packages/compass-components/src/components/accordion.tsx

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,14 +52,12 @@ interface AccordionProps extends React.HTMLProps<HTMLButtonElement> {
5252
hintText?: string;
5353
open?: boolean;
5454
setOpen?: (newValue: boolean) => void;
55-
onToggle?: () => void;
5655
}
5756
function Accordion({
5857
text,
5958
hintText,
6059
open: _open,
6160
setOpen: _setOpen,
62-
onToggle,
6361
...props
6462
}: React.PropsWithChildren<AccordionProps>): React.ReactElement {
6563
const darkMode = useDarkMode();
@@ -72,9 +70,6 @@ function Accordion({
7270
setOpenRef.current?.(newValue);
7371
return newValue;
7472
});
75-
if (typeof onToggle === 'function') {
76-
onToggle();
77-
}
7873
}, []);
7974
const regionId = useId();
8075
const labelId = useId();

packages/compass-indexes/src/components/create-index-form/create-index-form.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ function CreateIndexForm({
184184
<Accordion
185185
data-testid="create-index-modal-toggle-options"
186186
text={showIndexesGuidanceVariant ? 'Index Options' : 'Options'}
187-
onToggle={() => {
187+
setOpen={() => {
188188
track('Options Clicked', {
189189
context: 'Create Index Modal',
190190
});

0 commit comments

Comments
 (0)