Preview card made with flexbox

Community feedback

• 

  Hexer•3,660

  @EmmanuelHexerPosted almost 3 years ago

  Great work man, Nicely done

  • One tip i can give you is wrap the whole content with specific tag like main and attribution with footer tag.

  • always Include alt attribute in your img elements.

  Marked as helpful

  0
• 

  steezyza•90

  @steezyzaPosted almost 3 years ago

  This is a good attempt, well done.

  The only issues I noticed

  a) accessebility: Semantic HTML helps code readers navigate through the page easier e.g. <header> <main> <footer> and also include alt in your image tag e.g. <img src="./assets/images/image-avatar.png" alt="nft-card">

  b) the image should have a border radius e.g. border-radius: 5px;

  All the best for future solutions

  Marked as helpful

  0
• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Great work with this one. Regarding the solution you submitted, I think you should also take notes of the following…

  • Wrap the whole content with specific tag like main and attribution with footer tag. For HTML structures, refer it with this one Semantics
  • Include alt with description in image tags. Like this line <img src="./assets/images/image-avatar.png">
  • Add border-radius: 10px; in #equilibrium rule set. Apply also in the hover state

  Above all, The design looks good. Keep it up! Hope this is helpful!

  Marked as helpful

  0
• 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 📈😉

  • Add border-radius to the image also as you used in the card
  • You can add some box-shadow to the card too

  Happy coding ☕

  Maqsud

  Marked as helpful

  0