Design comparison
Solution retrospective
if you do the preview website the page is showing fine but i think there is some problem in my code because of that it is showing the three container stacking on over each other , feel free to give any suggestion .
Community feedback
- @gsterczewskiPosted over 3 years ago
Hello devansh 👋, my name is Grzegorz.
Regarding border on main container.
- Add
background-color
tobody
, you should have exact color in style guide - Try removing the
border: 1px solid grey
rule, but leaveborder-radius
. - Play around with
box-shadow
property - You don't need
border-bottom-right-radius
andborder-bottom-left-radius
on.right-container
, instead addoverflow-hidden
on.container
and it should do the trick.
I hope this will be helpfull to You.
Keep up the good work! 💪
See you on the coding trail 😉
Cheers!
Marked as helpful1 - Add
- @Chris94LeePosted over 3 years ago
Hey Devansh,
I believe the reason why it's showing the mobile design in the preview, is because of your media query:
@media (min-width:375px) and (max-width:1440px)
. So here you're using the 3 stacked containers on screen devices between 375px and 1440px. I'd recommend having the max-width as 1000px. And also to remove the min-width, because on devices smaller than 375px, it's reverting back to the web layout.If i'm incorrect here then someone else feel free to correct me.
0
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