-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Adding the "Open in Stackblitz" to the rest of components #18344
Comments
Hi, my team would like to work on this issue, could we get it assigned? |
Hey @JuanMacias153 If you want we can find other issues for your team to work! |
Hi @guidari, I would love to work on an the project, is there any "good first issue" ones that you are able to open up? My team are willing to contribute in any way that they can. Thank you so much! 🙂 |
You guys can grab anything in the queue that was already triaged. If you want an issue, just make a comment on it, so a Carbon member can assign you. |
@guidari @heloiselui I just want to be sure we're not losing track of Is the intent to revise playground stories and move args to all the default stories as part of this workstream or no? If so, it would be good to include that as part of the acceptance criteria for the PR(s) towards this issue. If not, we should work together to get a list going in #16816 of all the components that need their playground stories removed and args added to all the other stories. |
This PR added the "Open in Stackblitz" to all stories that had the
Canvas
already setup.In this issue it should be added the to the variants that don't have it. There is no need to add additional docs, the
<Canvas />
should be enough.Example:
Note: After adding the code, it should be tested by opening it in Stackblitz to ensure everything works as expected. Some stories might not function properly in Stackblitz, so we’ll need to evaluate whether they can be fixed or not.
The text was updated successfully, but these errors were encountered: