Design comparison
SolutionDesign
Solution retrospective
Feedback plz
Community feedback
- @Antoine-FloPosted about 4 years ago
Hi TheMihir 👋 good job, your design comparison looks nice.
Anyway, your layout isn't responsive right now, you have to find ways to fix your design so it doesn't move everywhere when the screen size changes.
- Float properties are nice but you should also use flexbox, to help you organize your elements with more control.
- Apply a
max-width
to your container so it doesn't stretch on big screens - I wouldn't use the
bg-pattern-desktop
as a background-image, but as animg
in your html, like the woman (with absolute positioning). - Notice that the 'woman' picture changes when in mobile size. You don't have to use the box anymore but the other illustration in you images folder.
Keep coding, layouts are challenging at first but it gets easier 👍
2@TheMihirSenseiPosted almost 4 years ago@Antoine-Flo thanks man really need this kind of feedback. that's actually means to give feedback.I will improve soon as possible thanks again.
1
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