-
-
Notifications
You must be signed in to change notification settings - Fork 41
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
feat: rounded box #348
feat: rounded box #348
Conversation
Run & review this pull request in StackBlitz Codeflow. |
Hi @JaimeTorrealba this is planned for v4 or would be better to have it available on an ealier version? |
I would say not for the v4. @alvarosabu |
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.
Hey @JaimeTorrealba ,
Looks good! Just some minor changes/cleanup.
docs/guide/shapes/rounded-box.md
Outdated
@@ -0,0 +1,30 @@ | |||
# Rounded-box <Badge type="warning" text="^3.8.1" /> |
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.
Probably best to use the component name as the page title:
# RoundedBox <Badge type="warning" text="^3.8.1" />
docs/guide/shapes/rounded-box.md
Outdated
@@ -0,0 +1,30 @@ | |||
# Rounded-box |
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.
(Non-blocking)
(I see that the badge was removed, but "Rounded-box" wasn't changed. I'm not sure if that was intentional.)
My preference would be to change this to ...
# RoundedBox
... since that's the name of the component. But there are some kebab-case names in the docs site, so I won't hold up the PR.
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.
Nop you're right, I'll fix this.
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. 👍 I approved the PR.
There's one comment to look at, but it's not blocking.
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.
Great!
No description provided.