Perfume Landing page using Html and Css
Solution retrospective
This one took me a around 5 hours. Surprisingly hard.
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@visualdenniss
Congrats on completing the challenge successfully! It looks great on both mobile and desktop devices and is responsive. However these is a suggestion i would like to make. The image seems to be distorted a bit, because of the aspect ratio of width and height. To handle this you can add object-fit:cover;
.imgdesktop { display: flex; height: 100%; object-fit: cover; }
Hope you find this feedback helpful!
Marked as helpful - @waffleflopper
I like how you formatted out your CSS styling, with the comments about what each was doing before writing all the layout styles and such. I'm an in-progress self-taught looking to start a career in the field in about a year and was wondering where you picked this method up? I'm always on the lookout to for new practices to implement =)
Nice work =)
Join 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