Skip to content
Submitted over 3 years ago

Desktop first HTML/SCSS 3 columns cards preview

Mohamed NDOYE•70
@mohamed-ndoye811
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...3-column preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

This was mainly for fun and quick practice but any advice or suggestion is heavily welcome!

Community feedback

  • tediko•6,680
    @tedikoPosted over 3 years ago

    Hello, Mohamed NDOYE! 👋

    Congrats on finishing another challenge! Your solution responds well. There's few suggestions from me:

    • Since your .card__icon image is decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies, such as screen readers.
    • Don't use divs for everything. Your .card__title should be a heading element <h1-h6> and your .card__text a paragraph <p>.

    Good luck with that, have fun coding! 💪

    0
    Mohamed NDOYE•70
    @mohamed-ndoye811Posted over 3 years ago

    @tediko Hey thanks for your answer! You're totally right, I'm going to make those changes thanks again for the advices!

    0

Please log in to post a comment

Log in with GitHub

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