dequelabs / dequelabs/cauldron

Accordion content props gets spread to multiple elements

Open
#714 1 comment 0 reactions 0 assignees View on GitHub
accessibility good first issue
Dominant language
TypeScript
Stars
127
Forks
31
Avg merge
2d 12h
Merged PRs (30d)
8

Description

```tsx
return (
setIsOpen(!open)}>
{label}
Here is some content

);
```

Then the rendered HTML has two elements with the same id, which is [invalid HTML](https://html.spec.whatwg.org/multipage/dom.html#the-id-attribute):

```html


...



Here is some content



```

Similarly, if I set any other attributes, such as `className` on the `AccordionContent` component, those attributes will be set on both the `ExpandCollapse__panel` and `Accordion__panel` elements.

_Originally posted by @thuey in https://github.com/dequelabs/cauldron/pull/713#discussion_r930402229_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.