Design comparison
Solution retrospective
It is the third challenge I have done! :D
I had some issues trying to keep the footer fixed at the bottom. If possible, I would appreciate your feedback about it and what else you see that I can improve!
Anyway, I really appreciate any help you can provide. ☺️
Community feedback
- @davidbdyerPosted almost 3 years ago
Each individual card aren't supposed to have rounded corners. It’s a little distracting that the whole card changes size when you hover over the button. The page is not responsive and breaks on mobile devices.
Your closing div was missing for the div opened on line 19.
Marked as helpful0 - @brodiewebdtPosted almost 3 years ago
As far as the Footer goes, it doesn't have to stick to the bottom of the screen. If you remove the position: fixed and the left and bottom values. It will be fine. I would also remove the max-height on the Car-Type Div as it is cutting off the content on mobile. Use Chrome DevTools set to Iphone X and you will see what I mean.
You can remove the aria hidden element on your Github link. You only want to use that for things like icon links.
Hope this helps.
Marked as helpful0
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