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

feat: improve event carousel animation #62

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

erichamilt0n
Copy link
Owner

Changes Made

  • Add seamless infinite scroll animation with proper cleanup
  • Implement edge fade effects for smooth transitions
  • Fix animation cleanup on component unmount
  • Add proper resize event handling
  • Optimize performance with useCallback

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Testing Completed

  • Verified smooth infinite scroll behavior
  • Tested resize handling
  • Confirmed proper cleanup on unmount
  • Checked edge fade effects
  • Validated performance optimization

Dependencies

  • No new dependencies added

Additional Notes

This improves the user experience by making the event carousel scroll smoothly and seamlessly.

- Add seamless infinite scroll animation
- Implement edge fade effects for smooth transitions
- Fix animation cleanup on unmount
- Add proper resize event handling
- Optimize performance with useCallback
@erichamilt0n erichamilt0n merged commit e382c4c into main Jan 17, 2025
2 checks passed
@erichamilt0n erichamilt0n deleted the feature/event-carousel-improvements branch January 17, 2025 17:48
Copy link

vercel bot commented Jan 17, 2025

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

Name Status Preview Comments Updated (UTC)
l2a-mobile-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 17, 2025 5:48pm

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.

1 participant