@mattstuddert
Posted
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!
@stevenCsaunders
Posted
@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?
@mattstuddert
Posted
@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.