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

v8 Rewrite #1043

Closed
wants to merge 69 commits into from
Closed

v8 Rewrite #1043

wants to merge 69 commits into from

Conversation

melvin-chen
Copy link
Member

@melvin-chen melvin-chen commented Feb 1, 2024

Nuka v8

Nuka v8 is an upgrade that aims to simplify the API and align the carousel with modern alternatives.

Highlights

  • Size reduced by more than 85% of the original package size
  • Much more responsive, don’t have to render two carousels and hide one with CSS for each breakpoint
  • Animations aren’t manually done and rely and browser/OS animations that are much more accessible, touch friendly, and result in a smaller bundle size
  • You can have different widths for each slide if needed
  • Voiceover accessibility is more intuitive for keyboard navigation users.
  • Documentation is improved with more in-depth examples and Typescript typing

Breaking Changes

  • Removed many props (with reason!). This is documented in the v8 Upgrade Guide page
  • Wrap around is not the same as it was before where the carousel would seamlessly flow from the end to the start again. It currently just scrolls backward to the first slide again.

Copy link

vercel bot commented Feb 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nuka-carousel-website ❌ Failed (Inspect) Mar 18, 2024 7:44pm

@carbonrobot
Copy link
Contributor

Closing this PR while this in progress so it doesn't keep triggering CI for no reason, will reopen when stable with a clean PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants