-
Notifications
You must be signed in to change notification settings - Fork 207
Components pages docs offsite overhaul #633
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
Conversation
content/components/accordions.mdx
Outdated
``` | ||
</Accordion> | ||
|
||
Check out [AccordionGroup](/content/components/accordion-groups) if you want to group multiple Accordions into a single display. |
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.
Can we move this to be below the props by chance?
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.
done in fb1487f
Co-authored-by: Hahnbee Lee <[email protected]>
Co-authored-by: Hahnbee Lee <[email protected]>
Co-authored-by: Hahnbee Lee <[email protected]>
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.
Can we add the ## Props
header above "AccordionGroup does not have props" just so it's all aligned in terms of how it's structured?
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.
done in 4465d06
This PR updates the pages in the Components section of the website
These pages are pretty nice already! Just added some missing props and a few more examples that I thought were clarifying
This PR also adds a separate page on reusable components from snippets, taking the content out of the snippets page