Skip to content
This repository has been archived by the owner on Jun 23, 2023. It is now read-only.

docs: Add new card component documentation #12

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

estheragbaje
Copy link
Member

Added documentation for the new card component

@vercel
Copy link

vercel bot commented Nov 15, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
chakra-ui-docs-next ✅ Ready (Inspect) Visit Preview Nov 15, 2022 at 1:12PM (UTC)

Copy link
Collaborator

@noobinthisgame noobinthisgame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for moving this over :) I've found only one thing that applies on all live previews


Put in some content in the `CardBody` to get a basic card.

```jsx live
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to check if the examples are rendered properly. I've added a way to make the live preview box different sizes so you don't have scrolling examples

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure I get what you mean. The examples render correctly with the live tag, although there's a scroll in the preview. In what way did you make the examples non-scrolling?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you check other components live blocks in the code. There is in some places sm,md,lg to describe which high it should have. This is because some components are to big for the default high of the iframe.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants