Skip to content

Conversation

@r3yc0n1c
Copy link
Contributor

@r3yc0n1c r3yc0n1c commented Dec 17, 2024

Description

  • fixed the Smooth Loop transition for venue cards
  • added Draggable Carousel for venue cards

Related issue(s)

Demo

draggable-carousal.mp4

@netlify
Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for peaceful-ramanujan-288045 ready!

Name Link
🔨 Latest commit 506fa2c
🔍 Latest deploy log https://app.netlify.com/sites/peaceful-ramanujan-288045/deploys/67f3c01e8a11e10008d7f09c
😎 Deploy Preview https://deploy-preview-496--peaceful-ramanujan-288045.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@AceTheCreator AceTheCreator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really awesome!

Do you think you can decrease the spacing between the venues cards. I think making them closer in mobile view will be aesthetically pleasing. wyt?

cc @ashmit-coder your thought will be appreciated here

@r3yc0n1c
Copy link
Contributor Author

This looks really awesome!

Do you think you can decrease the spacing between the venues cards. I think making them closer in mobile view will be aesthetically pleasing. wyt?

cc @ashmit-coder your thought will be appreciated here

Good idea! Yes, we can do that. Actually, I tried to keep the previous layout. Let me know what others think about the spacing. In the meantime, I'll continue working on the other issues.

@AceTheCreator
Copy link
Member

@r3yc0n1c, i noticed you didn't commit the spacing suggestion :)

@r3yc0n1c
Copy link
Contributor Author

apologies, I'll commit that ASAP

@r3yc0n1c
Copy link
Contributor Author

Before After
image image

@r3yc0n1c
Copy link
Contributor Author

@AceTheCreator let me know if this works

@AceTheCreator
Copy link
Member

@r3yc0n1c, kindly resolve the conflict in this PR

@r3yc0n1c
Copy link
Contributor Author

r3yc0n1c commented Mar 9, 2025

@r3yc0n1c, kindly resolve the conflict in this PR

done!

@ashmit-coder
Copy link
Collaborator

hey @r3yc0n1c please pull the latest changes and resolve the erros. Otherwise looks good to me @AceTheCreator

@AceTheCreator
Copy link
Member

hey @r3yc0n1c please pull the latest changes and resolve the erros. Otherwise looks good to me @AceTheCreator

@r3yc0n1c check why test failed

@AceTheCreator
Copy link
Member

reminder @r3yc0n1c

@animeshk923
Copy link

any update? @r3yc0n1c

@thulieblack
Copy link
Member

No response from the contributor, reassigning the issue

@thulieblack thulieblack closed this Dec 2, 2025
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.

5 participants