Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: carbon data table with radio button selection can not support multiple table in the same page #17190

Closed
2 tasks done
yaoterry opened this issue Aug 16, 2024 · 0 comments · Fixed by #17191
Closed
2 tasks done
Labels
component: data-table needs: community contribution Due to roadmap and resource availability, we are looking for outside contributions on this issue. role: dev 🤖 severity: 3 https://ibm.biz/carbon-severity status: help wanted 👐 type: bug 🐛

Comments

@yaoterry
Copy link
Contributor

Package

@carbon/react

Browser

Chrome

Package version

v1.63.2

React version

v17.0.2

Description

this is a change from #14644, it's required the consumer that one page having multiple tables with radio selection should pass the TableSelectRow name.
This is easy to be avoid from the library level, but not from the consumer application level, and the Table instance id can be supplied for that TableSelectRow name.

Reproduction/example

N/A

Steps to reproduce

our page has multiple tables, and it's failing for the radio button selection because they share a same name in different tables.

Suggested Severity

Severity 4 = Unrelated to a user task, has a workaround or does not need a workaround.

Application/PAL

No response

Code of Conduct

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data-table needs: community contribution Due to roadmap and resource availability, we are looking for outside contributions on this issue. role: dev 🤖 severity: 3 https://ibm.biz/carbon-severity status: help wanted 👐 type: bug 🐛
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants