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]: ExpandableTile does not forward a ref #13340

Closed
1 of 2 tasks
phtoussaint opened this issue Mar 13, 2023 · 1 comment · Fixed by #13390
Closed
1 of 2 tasks

[Bug]: ExpandableTile does not forward a ref #13340

phtoussaint opened this issue Mar 13, 2023 · 1 comment · Fixed by #13390

Comments

@phtoussaint
Copy link

Package

@carbon/react

Browser

Chrome

Package version

v1.17.0

React version

v18.2.0

Description

Looks like ExpandableTile, and maybe Tile too, does/do not forward a ref:

I set ref={ref}, on ExpandableTile (also tried on Tile) but ref.current is still "null"

Reproduction/example

https://stackblitz.com/edit/github-5hp6sj?file=src/App.jsx

Steps to reproduce

  1. get a ref with React useRef: const ref=useRef(null)
  2. set Component's ref like ref={ref}
  3. debug and check ref.current (i.e in a button handler function)
  4. ref.current is always null

Suggested Severity

None

Application/PAL

No response

Code of Conduct

@tw15egan tw15egan changed the title [Bug]: [Bug]: ExpandableTile does not forward a ref Mar 14, 2023
@tw15egan tw15egan moved this to ⏱ Backlog in Design System Mar 14, 2023
@pratikkarad
Copy link
Contributor

Hi, I would like to try my hands at this.

@tay1orjones tay1orjones moved this from ⏱ Backlog to 🚦 In Review in Design System Mar 23, 2023
@kodiakhq kodiakhq bot closed this as completed in #13390 Mar 30, 2023
@github-project-automation github-project-automation bot moved this from 🚦 In Review to ✅ Done in Design System Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants