Skip to content

fix: allow data-* and aria-* attributes - #344

Merged
afc163 merged 2 commits into
react-component:masterfrom
destimon:fix/rest-props
Mar 16, 2024
Merged

fix: allow data-* and aria-* attributes#344
afc163 merged 2 commits into
react-component:masterfrom
destimon:fix/rest-props

Conversation

@destimon

@destimondestimon commented Mar 15, 2024

Copy link
Copy Markdown
Contributor

To allow aria-* and data-* attributes we need to pass spread props inside of container as we're doing for other RC components as I see

@vercel

vercelBot commented Mar 15, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewCommentsUpdated (UTC)
collapse❌ Failed (Inspect)Mar 15, 2024 8:14pm

Comment threadsrc/Collapse.tsx Outdated
className={collapseClassName}
style={style}
role={accordion ? 'tablist' : undefined}
{...restProps}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Sure! Thank you, done

@codecov

codecovBot commented Mar 15, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.12%. Comparing base (18e931f) to head (9c18301).

Additional details and impacted files
@@ Coverage Diff @@## master #344 +/- ##
=======================================
Coverage 99.12% 99.12% =======================================
Files 5 5 Lines 114 114 Branches 40 40 =======================================
Hits 113 113 Misses 1 1 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@destimon@afc163