Submitted almost 3 years ago

3-Column previous module completed with HTML and SCSS

Jamie Moore• 230

@jamiem89

Preview site View code

This is a solution for...

3-column preview card component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

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

Raymart Pamplona• 16,140

@pikapikamart

Posted almost 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 your wrapper selector. width: 100% is much better since it occupies available screen width and vw 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 ^^

Abhishek Rawat• 690

@AbePlays

Posted almost 3 years ago

It look's amazing. However, there's some horizontal scrolling at smaller screen.

Jamie Moore• 230

@jamiem89

Posted almost 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!

Abhishek Rawat• 690

@AbePlays

Posted almost 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!

Please log in to post a comment

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