-
-
Notifications
You must be signed in to change notification settings - Fork 366
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: dynamic grid on new collection #5135
Conversation
✅ Deploy Preview for koda-nuxt ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
I would prefer not to add the size of the layout in query since this is user preference.
store the grid changes in vuex preferences
for now
updated, now under vuex |
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.
Noice!
✅ wfm
on this PR apply a dynamic grid on the new collection page |
@preschian And it is also applied to explore as shown in the screen record? Or does this apply only to collection? So far its looking good! edit: Oki, it makes sense now :D didnt see the another issue |
🚀 |
Code Climate has analyzed commit 93c549b and detected 0 issues on this pull request. View more on Code Climate. |
Thank you for your contribution to the KodaDot NFT gallery.
👇 _ 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 📸
Screen.Recording.2023-02-28.at.17.06.43.mov