-
Notifications
You must be signed in to change notification settings - Fork 109
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/upgrade carousel component to embla carousel #150
Feat/upgrade carousel component to embla carousel #150
Conversation
…de_carousel_component_to_embla_carousel
…arouselEmbla component.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@shuvo-me is attempting to deploy a commit to the StaticMania Team on Vercel. A member of the Team first needs to authorize it. |
Dear @shuvo-me, Your work is very good. You are to be commended for that. But we want to give you some guidelines how to make your work more beautiful and comfortable. We will use the embla-carousel library for the carousel component. For example:
You can put numbers in the carousel item, or use the pictures we have provided. It is completely up to you. Only the carousel should look like an instance of the embla carousel. I hope you understand. Best of luck. Thank you |
@Arifulislam5577 Thanks vai for your suggestions. I will update the features as well. |
0b46b3a
to
9ba9f41
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix this small issue. You are doing really well. After fixed this issue then I will merge your code in our main branch. Thank you.
https://www.loom.com/share/af6f8a1853364220ac167697387fb15e
@Arifulislam5577