Skip to content
Submitted over 3 years ago

Desktop-First HTML & CSS solution using CSS Flexbox

Boris•110
@borakb
A solution to the NFT preview card component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

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

Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • Sarang-Patel•190
    @Sarang-Patel

    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
  • Sahil•30
    @sahil905

    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
  • Renato Granados•160
    @Renato6GS

    Great job, I like it 👍

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