-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Also add ref to non-sortable branch in TableHeader #5204
Also add ref to non-sortable branch in TableHeader #5204
Conversation
Merge from carbon-design-system
Chore: get carbon-design-system/master
merge with sds-master
Deploy preview for the-carbon-components ready! Built with commit 4c22775 https://deploy-preview-5204--the-carbon-components.netlify.com |
Deploy preview for carbon-components-react failed. Built with commit 4c22775 https://app.netlify.com/sites/carbon-components-react/deploys/5e330e8cb9c25903ff3a3ce3 |
Deploy preview for carbon-elements ready! Built with commit 4c22775 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍 - Thanks @gmoehler!
BTW could you fix the code format? Thanks! |
Co-Authored-By: Josh Black <josh@josh.black>
@asudoh The react deployment probably needs a re-start |
Closes #4876 (well actually it is already closed we are still adding to it ;-) , maybe should re-open it?)
#4877 added forward refs to the
TableHeader
but did forget to add the ref to one codebranch. This PR is fixing this issue.@tay1orjones
@asudoh