@danielmrz-dev
Posted
Hello @Aaryan119!
Your project looks great and works well!
I just have a few suggestions for you to improve it:
-
Consider adding a
hover effect
to the numbers as well ascursor: pointer
to indicate that they are clickable. -
Take a look at the
style-guide
file to check what is the recommendedfont-family
for the project. -
Wrap the main content inside a
main
tag. This makes your HTML more semantic. UsingDiv
is not wrong, but they don't have semantic value. -
The color of the card is not a solid color, it's a gradient. You can create one by doing this:
background-image: linear-gradient(to bottom, color1, color2);
.
I hope it helps!
Other than those little details, you did a great job!
@Aaryan119
Posted
@danielmrz-dev hey thanks for the feedback I look into it thanks for giving feedback greatly appreciated 👍