Design comparison
Solution retrospective
Fun project, I'm new to HTML and CSS and found the complex parts with the responsive layout. Not really sure that I have got it right with the media query and positioning.
Community feedback
- @DrMESAZIMPosted over 2 years ago
Hi Ldybrk
I noticed that this is your first challenge which is very good attempt. Here are some issues i would like to help each other to make the solution look as close to perfection better
- Lets adjust the height of our container by making amendments to the file style.css
line 16 set the class "container " properties top this
.container { display: flex; justify-content: center; align-items: center; width: 70vw; height:; padding: 0.5em; margin: 20vh auto 0; }
- remove the height properties on this class
suv , sedans and luxury on lines 25 , 31 and 39 respectively
-
line 132 change the media screen properties to
@media screen and (max-width: 700px) {}
-
line 133 set " container" class properties to
.container { display: flex ; flex-direction: column; }
Marked as helpful0 - @LdybrkPosted over 2 years ago
Hi DrMESAZIM, Thanks a lot for your comment and advice. I have made the changes to the CSS file and posted it into my GITHUB repo. I have also had to change the bottom padding of the buttons. Thanks again, I am now moving forward to the next challenge. LDY.
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