HTML, CSS (FlexBox), Media Queries

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  Hey Steven, great work on this challenge! My only recommendation would be to add a h1 as the main heading as opposed to a h2. Every page you create should have a single h1 as this is then used as the main page title for screen reader users.

  Keep up the great work!

  0

  ---

  

  Steven Saunders•235

  @stevenCsaundersPosted almost 5 years ago

  @mattstuddert 👌.

  I have one additional question about using the section tag. Should you only use the <section> tag when you have a heading for the content in the tag? Is a div a better way to go if there isn't an h1-6 element?

  0

  

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  @stevenCsaunders hey Steven, yeah you should typically have a heading inside a section. So using a div instead would be fine. Sections are used for a larger grouping of themed content i.e. testimonial section or features/benefits section. So they're typically used for larger groupings of content which would normally include a heading.

  I hope that helps. Let me know if you have any other questions.

  0