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-feature-section-master

DONUZZ 140

@mrdonuzz

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Adriano 36,730

@AdrianoEscarabote

Posted

Hello DONUZZ, how are you? I was really pleased with your project, but I’d like to offer some advice that might help:

I noticed that you use more than one h1, this is not a good practice since it is recommended to have only one h1 per page to inform the main title of the page! so remove all h1 and put h2

<h1>Reliable, efficient delivery</h1>
<h1>Powered by Technology</h1>

The rest is spot on.

Hope it’s helpful to you. 👍

0
P

@laxmikishore13

Posted

you can use more semantic HTML in the code, with more pseudo-classes to give border-top. apart from that code looks clean and maintainable.

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