3_column_pre-view_card

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted about 2 years ago

  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 that SUVS, SEDANS, and LUXURY be <h2> elements.
  • The Learn More elements should be anchor 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

  0

  ---

  

  Aatif•90

  @AatifxShaikhPosted about 2 years ago

  @MelvinAguilar Thank You for the feedback. Will look into the tips and will try to improve more.

  1