@mattstuddert
Posted
Hey Tobi, congrats on submitting your first solution! You've done a really good job. Your code isn't very messy at all! After having taken a look I've got a few pointers that will hopefully help:
- For your
. heading
class, I would recommend not using asection
here. Asection
is typically used to group a larger collection of themed content. For example, a testimonials section on a page. For this smaller grouping of content, adiv
would be absolutely fine. - You've got
h1
andh3
headings but noh2
. Be sure to avoid skipping heading levels unnecessarily, as this can cause accessibility issues. - You don't need the
div
wrapping the anchor tag. You could just style the anchor tag directly. - In the Why Us area, I'd use a
ul
withli
elements to create a list instead of having separatep
tags for each item.
You've done a really good job. Let me know if you have any questions 🙂
@Tobi-dev99
Posted
@mattstuddert Wow,that was very helpful! Thank You for sparing some time and effort to review my solution.Those were really some pointers.I'll keep those in mind when taking on next challenge.
@mattstuddert
Posted
@Tobi-dev99 you're welcome, Tobi! Happy to help 🙂