Design comparison
Solution retrospective
Hello again :)
I think this is not the best "responsive" implementation (if it is 😬), so I appreciate any reviews I get.
Thanks in advance.
Community feedback
- @reza-sjdnPosted over 1 year ago
Hi @OoMiDOoO, Good Job!
Your work at mobile view is great but at desktop view is a little bad. Set
width: 50%
andheight: 100%
for the#perfume-img
. This way it will likely be ok.but anyway Keep going your journey!
1@G0maaPosted over 1 year ago@reza-sjdn
Thanks for the feedback and help! I will play around the solution a bit till it looks better.
0 - @BimboAdesoyePosted over 1 year ago
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.
1@G0maaPosted over 1 year ago@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 :)
0
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