@Rubylenshy
Posted
Hi, @mohamedshawgi ๐ Congratulations on completing this challenge. Here are a few tips to improve:
- On this declaration .sedan,.suvs,.luxury{height: 80%;}, that height would give you an offset height which I think would get too big at some point. You could just add padding to the cards .sedans, .suvs, .luxury{ padding: 40px; } Hence you won't need this { height:auto; max-height: 650px; }
- And your .container could be position centered just like .container{ position: absolute; top: 50%; left: 50%; transform: translate(-50%, -50%) } Hence, the {width: 55%;} wouldn't matter
Note: This might not be a perfect correction but it could guide you
Great Work @mohamedshawgi, Keep Coding
Marked as helpful