Design comparison
SolutionDesign
Solution retrospective
All the feedback is welcome thank you in advance!
Community feedback
- @danielmrz-devPosted 11 months ago
Hello @OsmanSamar!
Your project looks great!
I have a few suggestions that might interest you:
- You can replace that
h2
withh1
since it's the main title on the screen.
The
<h1>
to<h6>
tags are used to define HTML headings.<h1>
defines the most important heading.<h6>
defines the least important heading.Only use one
<h1>
per page - this should represent the main heading/subject for the whole page. Also, do not skip heading levels - start with<h1>
, then use<h2>
, and so on.- Also, you might wanna change the
.card-description p
color to a lighter one. Currently it's too dark, and since the bakcground is dark too, it's almost impossible to see it.
I hope it helps!
Other than those little details, you did a great job!
Marked as helpful0 - You can replace that
Please log in to post a comment
Log in with GitHubJoin 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