Design comparison
Solution retrospective
Any feedback will be helpful
Community feedback
- @VCaramesPosted about 2 years ago
Hey @AfricanBambatha69, some suggestions to improve you code:
- To center you content to your page, add the following to your Body Element:
body { min-height: 100vh; display: grid; place-content: center; }
-
The car images/icons serve no other purpose than to be decorative; They add no value. Their Alt Tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.
-
The headings are being use incorrectly. The <h1> heading is only allowed to be used one per page. So the best option, is to use <h2> Heading, because it will give each card the same level of importance and it's reusable.
-
Your "buttons" were created with the incorrect element. When the user clicks on the button they should directed to a different part of you site. The Anchor Tag will achieve this.
Happy Coding! 👻🎃
Marked as helpful0@AfricanBambatha69Posted about 2 years ago@vcarames Noted that was alot of good stuff thank you
0 - @AdrianoEscarabotePosted about 2 years ago
Hi wani, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
- every Html document must contain the main tag, so we can identify the main content, to fix this, wrap all the content with the
main
tag. - To align some content in the center of the screen, always prefer to use
display: flex;
it will make the layout more responsive!
Example:
body { margin: 0; padding: 0; display: flex; align-items: center; justify-content: center; min-height: 100vh; }
The rest is great!
I hope it helps... 👍
Marked as helpful0 - every Html document must contain the main tag, so we can identify the main content, to fix this, wrap all the content with the
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