-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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(components): Create sidebar component family #6109
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Appreciate the PR. This one has been open for a couple of weeks now and looks like it's almost ready to land: #6065 . Perhaps you could collaborate there if you have any suggestions. |
yes I think we can close it. I agree with Mike that though the timing wasn't right this time, we really welcome folks contributing however they can, even if it's a stray PR review or issue report. Every bit helps! |
Description
This PR introduces changes for issue #6061. Code features new
Sidebar
,SidebarGroup
andSidebarItem
components.Validation
Related Issues
Resolves #6061
Check List
npx turbo lint
to ensure the code follows the style guide. And runnpx turbo lint:fix
to fix the style errors if necessary.npx turbo format
to ensure the code follows the style guide.npx turbo test
to check if all tests are passing.