Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Desktop-First HTML & CSS solution using CSS Flexbox

Boris 110

@borakb

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I would like feedback on how I can improve my solution.

Community feedback

@Sarang-Patel

Posted

Congratulations on your solution.

Just a few things to note:

  1. The "Equilibrium #3429" has hover effect even outside the content. To fix that set its width to fit content in css.
  2. Use alt attribute for images.
  3. Contain all the code in body tag in main tag.
  4. Use One h1 tag.

points 2,3,4 will help in removing accessibility issues.

Hope this helps.

Marked as helpful

1
Sahil 30

@sahil905

Posted

Great design. Very neat code and totally matches the design. The only suggestion I can see is you can have a "font-weight: 600;" on your price element (.container ul li:first-child). 👍

Marked as helpful

1

@Renato6GS

Posted

Great job, I like it 👍

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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