-
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
Types/tabs #13635
Types/tabs #13635
Conversation
✅ Deploy Preview for carbon-components-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
This is looking good! Behavior works as expected, just a couple change requests
@kubijo thanks again for this! There's some remaining feedback to address here. I'd like to get this in but if it's stalled out we'll close after a few days. |
As far as I can tell, someone moved a lot of code here and as a result, we're getting MANY new errors in the Is there any chance of moving this to the author of those changes (them having the context of what was changed) and using this as a source of entropy? |
@kubijo sorry for the churn, There's been some new features added to Tabs recently, but I think it would only impact typings for two new props, |
Can someone on your side do the rebase, then? You (as in maintainers) should have a write access, I believe. |
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.
🚀
Thanks for your help @francinelucca! |
Closes #13540, #13541
Changelog
New