HTML & CSS Only

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted over 4 years ago

  Nice work on this challenge! Here are some pointers after taking a look at your code:

  • You've got h2 and h4 headings but no h1 or h3. I'd recommend avoiding skipping heading levels unnecessarily. This can cause accessibility issues in the content hierarchy for screen reader users. So the main heading would be a h1 and the next level in the hierarchy would be h2 and so on.
  • In the "Why Us" region you've got a single paragraph of text. I'd say this is a list of features/benefits so a ul would be more appropriate.
  • Avoid using IDs as CSS selectors. They have high specificity and can't be reused on the page, so they're not good for the purpose of styling. Instead, I'd recommend sticking to class, attribute, pseudo, and type selectors. Using these will help keep your CSS more maintainable.

  I hope this helps. Let me know if you have any questions!

  1