Hello again :)
I think this is not the best "responsive" implementation (if it is 😬), so I appreciate any reviews I get.
Thanks in advance.
@reza-sjdn
Posted
Hi @OoMiDOoO, Good Job!
Your work at mobile view is great but at desktop view is a little bad. Set width: 50% and height: 100% for the #perfume-img. This way it will likely be ok.
but anyway Keep going your journey!
@G0maa
Posted
@reza-sjdn
Thanks for the feedback and help! I will play around the solution a bit till it looks better.
@BimboAdesoye
Posted
Hello, this looks quite good. Although if you could find a way to make the picture fit its container entirely, that would be fantastic. I think it's important to always strive to get the solution as close as possible to the design.
@G0maa
Posted
@BimboAdesoye
Thanks for the feedback!
Yes, you are correct, it is best to get the solution as close as possible to the design.
I noticed the image problem and I'm working on it, any hints or suggestions of where to search is always appreciated :)
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
