-
-
Notifications
You must be signed in to change notification settings - Fork 73
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: carousel component using embla carousel #527
base: main
Are you sure you want to change the base?
Conversation
👷 Deploy request for kobalte pending review.Visit the deploys page to approve it
|
@Waishnav Might want to check if using From what I understand, UI primitive libraries like this one will generally avoid adding dependencies. |
Hmm make sense |
Thanks for the draft! I think this could be a good component, rewriting an entire carousel is out of scope for now. The API should ideally be fully wrapped in Kobalte with props on the Looking forward to this, good luck! |
through options and plugins props to Root of Carousel
Hey @jer3m01, I've implemented all the suggestions from your previous message. The With Kobalte's WAI-ARIA compliance and unstyled component philosophy, users can now compose their own custom carousel components as needed. That said, I’m still relatively new to writing composable UI components in this pattern. I’d greatly appreciate it if you could review it thoroughly and share any feedback or suggestions for improvement. Looking forward to iterating further based on your input! |
one more thing, should we add the keyboard interaction support just like we have for |
No description provided.