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

First Challenge I have tried - Four Card Feature

@TheInstantiator

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


First time I have tried to do anything after taking a course on udemy. Let me know how I did and what I could do better.

Thanks

Community feedback

Darryl 155

@DJH-dev

Posted

Great Job the site looks good and the code is nice and readable. Only thing I would add is maybe trying to use some of the HTML5 semantic elements like header and section instead of div in some places but that is subjective. Great Job

2

@krebeDev

Posted

Your work is clean and beautiful. However, in addition to the HTML5 semantic elements already suggested by DJH-dev, you should have used tags like headings (h1,..h6) and <p> for accessibility.

1

@TheInstantiator

Posted

Thanks all for the feedback, after I submitted and took another look, I did think to myself why I didn't use any headings and p tags.

0

Account Deleted

just complete all challenges and u will see the difference .. looks great work bro ..

0
P
Matt Studdert 13,611

@mattstuddert

Posted

@DJH-dev and @krebeDev have given some great feedback, so I've got nothing more to add other than to say well done and congrats on submitting your first solution! 🙂

0

@TheInstantiator

Posted

Thanks for the feedback, I will take a look at those elements.

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