@DivineUgorji
Posted
Hello @om__prabhat, congratulations on completing and submitting this challenge. Your implementation looks really nice but I could not help but notice that your background image is not well positioned. Apply the following properties to your code:
background-color: hsl(224, 100%, 94%);
background-image: url(image-url);
background-position: top;
background-repeat: no-repeat;
- I would also suggest the use of BEM methodology for your CSS naming conventions.
- I observed the overuse of the pixel unit in your code base. It's always good practice to learn how to use relative units in your projects as it helps with responsiveness. You can check out this blog to learn more about this.
overall, your implementation is very nice, and I hope these few suggestions of mine will help you to improve it better.
All the best 🙏
Marked as helpful
@om-prabhat
Posted
Thank you for giving feedback. I will surely look it out.@DivineUgorji