-
-
Notifications
You must be signed in to change notification settings - Fork 361
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 for resources on gallery detail #5751
Conversation
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
AI-Generated Summary: This pull request introduces a carousel feature for the gallery component. It modifies three files, |
@exezbcz while waiting expected design use this temporarily? |
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.
It looks strange that there is only one dot there.
https://deploy-preview-5751--koda-canary.netlify.app/ksm/gallery/13328844-54bbd380dc3baaa27b-EVRSOULS-Alchemist-00000783
@preschian yup, works |
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.
seems like there is no hover style even on canary or production? |
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.
otherwise lgtm
yes it do have the fullscreen button, i added it lol and this iShot_2023-04-23_13.25.36.mp4 |
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.
lgtm
@preschian there was a service outage |
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.
otherwise lgtm
yes. seems |
…sel-for-resources
oohh, sorry I misread this, I thought it was about hover style. added now |
functionality good - i will follow up with the stylign |
Code Climate has analyzed commit 311f491 and detected 0 issues on this pull request. View more on Code Climate. |
pay 70 usd |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
pay 70 usd |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type
Context
Before submitting pull request, please make sure:
Optional
Had issue bounty label?
Community participation
Screenshot 📸
Copilot Summary
🤖 Generated by Copilot at f754fbe
This pull request enhances the gallery item component to show the additional resources of the NFT in a carousel. It introduces a new custom type
NftResources
to represent the resources, and updates theuseGalleryItem
anduseNft
composables to handle the new type. It also modifies theGalleryItem.vue
component to use the carousel and hide the fullscreen button if there are resources.🤖 Generated by Copilot at f754fbe