Design comparison
SolutionDesign
Solution retrospective
The main challenge which I felt was to apply the tint cyan on the image, however I manage to do it by having an absolute div inside the image-container and giving it an hsla color of cyan. There may be better method of doing it, Hope to get feedbacks from mentors, Happy coding :-> Thank You.
Community feedback
- @faha1999Posted about 2 years ago
Hello saurabh, Congratulations on finishing this project. It's lovely and great on the whole! Just a little tip:
- remove all
margin
fromcontainer
class because with flex they are superfluous - instead of using
px
use relative units of measurement likerem
read here => - You could also make use of
<section>
and/or<article>
semantic tags. This would help improve accessibility.
I hope this helps you. Happy Coding!
Marked as helpful0@akgec-saurabhPosted about 2 years ago@faha1999 Thank You for your lovely suggestion I will surely implement it
0 - remove all
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