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

Four card layout using flexbox

@adetona54

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


All feedbacks are welcome

Community feedback

MarcoDV 280

@MarcoDV47

Posted

Hey, it looks nice! Some tips i can help you with:

HTML

  • span heading: two heading tags would be better here, they give more meaning and accessibility :
<span class="head-1"> Reliable,efficient delivery </span>
 <span class="head-2">Powered by technology  </span>

CSS

  • header: instead of using flex to center text you could use text-align on the header tag.
  • <img> alt tags: as they're decorative you can leave the alt="" blank.

Other than that it's a really great job! I liked your decision to go with flex. Maybe try with Grid some other time and compare to see what you feel more comfortable with.

Marked as helpful

0

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