@vanderms
Posted
Hi nice work!
Your solution is great and overall I liked it very much.
About the width and height, you could have used a more straightforward approach. I mean, if you needed the card on desktop viewports to have 576px, you should have set the div.main-container to have this width.
But in your solution, you set the body to have 1440px and the div.main_container to have 40% of that. And this is a problem, because it just works if the desktop has exactly this size. For example, on laptops 1280px wide, your site has a horizontal scrollbar because the <body> is overflowing and on desktops with 1980px the card is not in the center.
I think also that you should have used more semantic html tags. I mean, the .main_container could be an article (or a section) not a div.
Marked as helpful