@danielmrz-dev
Posted
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 helpful