-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
fix: collection on mobile is broken #5473
Conversation
SUCCESS @Jarsen136 PR for issue #5448 which is assigned to you. Please wait for review and don't hesitate to grab another issue in the meantime! |
✅ Deploy Preview for koda-canary 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.
For 2 columns struct, the text would overflow out of the screen. I guess placing them in one column on mobile could contain more text content. WDYT @exezbcz |
@Jarsen136 yes. that is better, I made it like that to make it simpler to convert the desktop design to mobile. In this case I dont mind that, but thanks for noticing! @daiagi |
@@ -19,15 +19,15 @@ | |||
@click.native="toggleSeeAllDescription" /> | |||
</div> | |||
<div> | |||
<div class="columns is-mobile"> | |||
<div class="column"> | |||
<div class="columns pt-3"> |
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 wouldn't add padding on columns/column, wrap it in a new div instead
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.
In this component, I need to overwrite some padding on column
, so I guess I could not wrap it in a new div. wdyt
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 don't add spacing in columns/column class. If you want to add spacing then create a inner div that wrap content.
If you don't want spacing you can use gapless helper or change columns to flex.
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.
Thanks, I have changed columns to flex.
Sooo last fixes and LG (let's go/ life's good)(choose for yourself) 🚀 |
Code Climate has analyzed commit b96c047 and detected 0 issues on this pull request. View more on Code Climate. |
pay 30 usd |
😍 Perfect, I’ve sent the payout 🪅 Let’s grab another issue and get rewarded! |
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 📸