Design comparison
Solution retrospective
Feedback welcome. on the screenshot generated here if you look at the background-image bottom right you can see its not completely on the bottom; but on my screen it is. So i dont understand why the screenshot shows it like this. Maybe preview the site on your screen and tell me what you see, especially if you have 1440 screen what do you guys think about fixed widths ? Thanks very much
Community feedback
- @MelvinAguilarPosted 10 months ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- It's a bit unusual to use a media query with such a high breakpoint to change the max-width. According to some solutions I see, it's very common to set these styles directly on the container:
max-width: 1270px; margin-inline: auto
. I also feel that 1270px is too large of a breakpoint for switching between desktop and mobile screens.
- Regarding the issue of not seeing the background image until the bottom, it's because the element doesn't have a height. Without height, the image won't know the
bottom right
position. As mentioned, usingmin-height: 100vh
will fix this.
I hope you find it useful! ๐
Happy coding!
Marked as helpful1 - It's a bit unusual to use a media query with such a high breakpoint to change the max-width. According to some solutions I see, it's very common to set these styles directly on the container:
- @danielmrz-devPosted 10 months ago
Hello @mbd89!
Your project looks great!
I have just one suggestion:
- There's a vertical scrollbar appearing on your project. If you remove all the body
padding
and addmin-height: 100vh;
to it, the scrollbar disappears and your project will be centered.
The screenshots are not always completely accurate, so I wouldn't worry about that. On my screen, your project looks excellent!
I hope it helps!
Other than that, you did a great job!
Marked as helpful0 - There's a vertical scrollbar appearing on your project. If you remove all the body
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