Gonzalo M. Núñez• 110
@gnunez0101
Posted
Hello Farhan Ali,
I took a look to your solution and seems really good. I have some comments about it:
- You used Calibri font for the solution. Looks good, but, the requested font was Outfit. Remember, for frontend development attention to details are everything. Use the requested font to get a closer appereance to design.
- It would be nice to use CSS custom properties. At least for the colors. This will make code more reusable.
- If you are using Flexbox, prefer
align-items
andjustify-content
for centering items instead oftext-align: center
. They work better and never fail. - You are centering vertically with
padding
andmargin
. Usealign-items
for better results. - For next solutions replace the link in "Coded by" with your profile's link. You left the
#
from the template.
I hope this notes can help you improve your next challenges.
Have a nice coding,
Gonzalo.
0