-
Notifications
You must be signed in to change notification settings - Fork 807
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
doc(toggle): Update the Usage tab with the latest content template #4440
doc(toggle): Update the Usage tab with the latest content template #4440
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Could a dev help re-deploy this PR so we can view the deploy preview? Thanks! |
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.
Looks great @Kritvi-bhatia17! 🥳
I have just suggested some minor changes. Ignore if irrelevant!
- introducing "a/the" wherever i found it missing.
- one tiny spacing issue in the anatomy legend.
- suggested capital letter for "s" in "state text" in the content section.
Hey @annawen1, would you be able to fix the vercel deploy preview when you get a chance? Thanks! |
Hi @alisonjoseph, @guidari, can you please help with the preview link. Thanks! |
Here it is the preview link: carbondesignsystem-git-fork-kritvi-d77586-carbon-design-system.vercel.app If it keeps failing and you are unable to merge, let me know. I can merge with the preview failing (before I'll test locally the build to make sure everything works fine and the problem is on Vercel) |
Thanks a lot @guidari! |
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.
Looks good! 👍
Hi @alisonjoseph, @annawen1, can you please review this PR when you get a chance? |
Closes #4215
Update the Usage tab with the latest content template and fixed a few more sections.
Changelog
New
Changed