column-preview-card-component-main

Community feedback

• Account deleted

  Hi @lewmas9152

  Congratulations on completing the challenge.

  You should try move away from using <div> so much and use semantic HTML. Landmarks are important for adding structure to your page and landmarks such as <header>, <main>, <nav> and <footer> aid navigation in assistive technologies such as screen readers and for keyboard accessibility. You should use <main> for your cars container and then <section> to wrap the individual car classes. Pages always include a <h1> and it is not recommended to skip header levels but you can have multiple <h1> on a page. Instead of <button> you should have used anchor tag <a> and styled it to look like a button. <button> is intended for actions carried out on the page whereas in this case the learn more button would take you to another page and is therefore a link.

  I hope this helps.

  Marked as helpful

  1