Design comparison
Solution retrospective
hey guys this was a very hard thing for me to do I would like every feedback you guys give there is a problem when you start resizing since I wanted to include a transition for the images I had to use translate so the problem is about translate coz it doesn't resize when start shrinking the page using window.onresize works but it make the performance much worse so I did not include it PLZ LET ME KNOW IF THERE IS A SOLUTION TO THIS THX IN ADVANCE just refresh the page when you resize and everything should work just fine :)
Community feedback
- Account deleted
This is truly not a simple challenge,
At 1440px it looks fine, works well, but the text is not changing, I'm not sure if that was purposeful or not. Just as you down-size the window, I can see the other picture peeping on the side, and this goes on, until no picture is in sight. Also at mobile view I see two pictures, and then no picture when I click.
I think you should re-do this one, that's how we all get a bit better.
If you like, you may check out my solution, and I'm not saying it's perfect but it works; https://www.frontendmentor.io/solutions/room-home-page-mobile-first-scss-and-vanilla-javascript-8lBZjmZov
1@Mohammd1321Posted over 3 years agobtw thx for the feed back and the heading size does become smaller but the h3 does not I don't feel it need to
0Account deleted@Mohammd1321
Until it happens again, and then I'll have to keep refreshing.
1@Mohammd1321Posted over 3 years agowell i kinda don't know how to fix it tbh with you and I don't think ppl use phones or desktops that can scale so it should work for every device that its in 😅 idk how to fix it tbh
0
Please log in to post a comment
Log in with GitHub
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