@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
- You are overusing the
section
element. A simplediv
will do.
- There are only three headings in this component; “Join our community”, “Monthly Subscription”, and “Why us”. Everything else would be wrapped in a
paragraph
element.
- The button was created with the incorrect element. When users click on the button they should directed to a different part of your site; the
anchor
element will allow this to happen.
More Info:📚
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!🎄🎁
Marked as helpful
@nerdy-guy
Posted
@vcarames Thanks for your feedback. Regarding <section>
aren't they better than div
for semantic and accessibility? Should I replace all of them with div?
@VCarames
Posted
@nerdy-guy
The section
element really has no semantic meaning. It is the same as a div
. the only difference is that the section
element is used to define groups of related content.
So for this challenge you "could" wrap the entire component in a section
element since they are all related.
Marked as helpful