Responsive NFT Preview Page using Flex mostly for Alignment.
Design comparison
Solution retrospective
ANY feedback would be appreciated!
Community feedback
- @NaveenGumastePosted almost 3 years ago
Hay ! EOSPryce Good Job on challenge
<div class="nft-description-wrapper"> <h3>Equilibrium #3429</h3>
Here you have to use
h1
tag-> Always use
h1
first and thenh2
,h3
and so on-> Check my article on accessibility issues
If this comment helps you then pls mark it as helpful!
Have a good day and keep coding π!
Marked as helpful2 - @denieldenPosted almost 3 years ago
Hi EOSPryce, I took some time to look at your solution and you did a great job!
Also try to add a little
transition
on the element with hover effectOverall you did well :)
Hope this help and happy coding!
Marked as helpful1 - @MubeenAhmadShaikhPosted almost 3 years ago
Looks good, you can add h1 tag instead of h3 for that accessibility issue.
Marked as helpful1 - Account deleted
Hi there π
Congratulate on finishing your project π. You did a great job π‘
I give some suggestions to help you take your project design to the next level ππ
- Change the
h3
font-weight from500
to600
to make it bolder π - Also make 0.041 ETH font-weight bolder π
- Use this code to add box shadow to the card
box-shadow: 0 12px 24px rgb(0 0 0 / 10%);
Happy coding β
Maqsud
Marked as helpful1 - Change the
- @ToniHunter274Posted almost 3 years ago
Looks good to me..Keep it up!π
Marked as helpful1
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord