Skip to content
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

📝 docs(side tabs): Add sui-side-tabs with radio to showcase. #521

Open
wants to merge 2 commits into
base: version/2.12.11
Choose a base branch
from

Conversation

creador-dev
Copy link
Member

No description provided.

@creador-dev creador-dev changed the title 📝 docs(side tabs): update documentation for side tabs radiobox 📝 docs(side tabs): Add sui-side-tabs with radio to showcase. Jul 13, 2022
@iamleigh iamleigh changed the title 📝 docs(side tabs): Add sui-side-tabs with radio to showcase. docs(side tabs): Add sui-side-tabs with radio to showcase. Jul 14, 2022
@iamleigh iamleigh added the documentation Additions or modifications to the showcase or docs label Jul 14, 2022
Copy link
Contributor

@iamleigh iamleigh left a comment

Choose a reason for hiding this comment

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

@creador-dev, you are replacing the current demo markup with new content, and that's not what we need. Please, provide in the documentation both case scenarios:

  • Tabs with buttons only.
  • Tabs with radio inputs.

@creador-dev creador-dev changed the title docs(side tabs): Add sui-side-tabs with radio to showcase. 📝 docs(side tabs): Add sui-side-tabs with radio to showcase. Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Additions or modifications to the showcase or docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants