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

[BUG] - Some components are missing Slots section in their documentation. #3233

Closed
aayush-goyal opened this issue Jun 11, 2024 · 3 comments · Fixed by #3580
Closed

[BUG] - Some components are missing Slots section in their documentation. #3233

aayush-goyal opened this issue Jun 11, 2024 · 3 comments · Fixed by #3580
Assignees
Labels
📋 Scope : Docs Related to the documentation

Comments

@aayush-goyal
Copy link

NextUI Version

2.4.1

Describe the bug

A few components are missing Slot section on their documentation.

Here is the comprehensive list of such components:

  1. Badge
  2. Card
  3. Switch

Your Example Website or App

No response

Steps to Reproduce the Bug or Issue

  1. Go to https://nextui.org/docs/components/
  2. Click on any of these components: a) Badge b) Card c) Switch
  3. Check right navigation bar for the content.
  4. Now click on Popover component.
  5. You will find Slots section for this component.

Expected behavior

As a user, I expected that if a component is making use of Slots and classNames, there should be proper documentation on that.

Screenshots or Videos

Screenshot 2024-06-11 at 1 13 11 PM
Screenshot 2024-06-11 at 1 14 24 PM

Operating System Version

macOS

Browser

Chrome

Copy link

linear bot commented Jun 11, 2024

@wingkwong wingkwong added the 📋 Scope : Docs Related to the documentation label Jun 11, 2024
@aayush-goyal
Copy link
Author

I can pick this up @wingkwong

@wingkwong
Copy link
Member

@aayush-goyal thanks. go ahead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📋 Scope : Docs Related to the documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants