Skip to content
This repository was archived by the owner on Feb 13, 2026. It is now read-only.

ui v2: accordion group#754

Merged
danielhochman merged 3 commits intoUIV2from
ui-v2-accordion-group
Dec 7, 2020
Merged

ui v2: accordion group#754
danielhochman merged 3 commits intoUIV2from
ui-v2-accordion-group

Conversation

@danielhochman
Copy link
Collaborator

Description

Add a mutually exclusive expanded accordion group.

Testing Performed

Storybook.

@danielhochman danielhochman requested a review from a team as a code owner December 7, 2020 18:29
};

export interface AccordionGroupProps {
children: React.ReactElement | React.ReactElement[];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
children: React.ReactElement | React.ReactElement[];
children: React.ReactElement;

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This JSX tag's 'children' prop expects a single child of type 'ReactElement<any, string | ((props: any) => ReactElement<any, string | ... | (new (props: any) => Component<any, any, any>)>) | (new (props: any) => Component<any, any, any>)>', but multiple children were provided.ts(2746)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(worked for node but not for element)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah sorry I misread this as ReactNode

@danielhochman danielhochman merged commit 5e9f46c into UIV2 Dec 7, 2020
@danielhochman danielhochman deleted the ui-v2-accordion-group branch December 7, 2020 22:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants