Evandro
@evandromarisAll comments
- @dzenitaa96Submitted about 2 years ago@evandromarisPosted about 2 years ago
Hallo Dzenita!
Excellent work, on desktop and mobile version.
Just an observation, remove of max-height: 30px; of the .user class, this way the images will be round, e uma dica.😊
💡A tip, always check the accessibility of HTML right here on Frontend Mentor, to always improve the semantics of HTML for search engines.
Good Job !🏅
Marked as helpful0 - @dan-ayalahdezSubmitted over 2 years ago
Could anyone give me some input on why does the design looks good on desktop, but if you switch to mobile its not aligned? I could not find a way to fix it
Thanks!
@evandromarisPosted over 2 years agoTry using 100vh in the height property of your body, that way you use 100% of the viewport, in this case as it's an NFT the size of a cell phone, it can be an alternative and simple fix.
body { ... height: 100vh; ... }
Hope this helps :)
In addition to this fix, you can check your code report here on Frontend Mentor, so you can always improve accessibility and HTML code.
https://www.frontendmentor.io/solutions/solution-for-the-nft-card-project-Z6cf2CD6Ti/report
1