@MelvinAguilar
Posted
Hi @AatifxShaikh ๐, good job on completing this challenge! ๐
Here are some suggestions you might consider:
- The
<h1>
element is the main heading on a web page. There should only be one<h1>
tag per page. The HTML Section Heading elements (Reference). This case requires thatSUVS
,SEDANS
, andLUXURY
be<h2>
elements. - The
Learn More
elements should beanchor
tags and not buttons because they redirect to another part of the page. - The container isn't centered correctly. You can use flexbox to center elements:
body {
min-height: 100vh;
display: flex;
flex-direction: column;
justify-content: center;
align-items: center;
}
Also, remove the margin from the container
selector.
Links with more information:
- The Complete Guide to Centering in CSS.
- A Complete Guide to Flexbox (CSS-Tricks).
- How TO - Center Elements Vertically (W3Schools).
- CSS Layout - Horizontal & Vertical Align (W3Schools).
I hope those tips will help you.
Good job, and happy coding!
Marked as helpful
@AatifxShaikh
Posted
@MelvinAguilar Thank You for the feedback. Will look into the tips and will try to improve more.