3-Column previous module completed with HTML and SCSS
Design comparison
Solution retrospective
This was quite a simple one. Pretty happy with this. I didn't really pay any attention to tablet, so just judge on mobile and desktop!
Community feedback
- @pikapikamartPosted over 3 years ago
Hey, great work on this one. Layout both desktop and mobile is good and responds very well.
A little suggestions would be that:
-
Like what AbePlays said, there is this scrollbar. You can fix this by removing the
width: 100vw
on yourwrapper
selector.width: 100%
is much better since it occupies available screen width andvw
doesn't really account the edges or notch. -
Adding a padding to the top and bottom of the body so that when inspected, it doesn't touch the ceiling and flooring of the screen.
Really good job on this ^^
1 -
- @AbePlaysPosted over 3 years ago
It look's amazing. However, there's some horizontal scrolling at smaller screen.
1@jamiem89Posted over 3 years ago@AbePlays Which browser are you using? I had this comment on another of my solutions but I'm not getting it on any of the major browsers using mac osx. Any info would really help me out!
0@AbePlaysPosted over 3 years ago@jamiem89 I primarily use Brave but i checked it on Firefox too. There is definitely some horizontal scrolling. Try what pikamart suggested in the next comment. See if that fixes the issue. Cheers!
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