Skip to content
OverviewSolutions
5
Comments
2
Andrew
@L1m1tz

All comments

  • Andrew•90
    @L1m1tzPosted over 2 years ago

    Hey, Great Job completing your first Frontendmentor challenge. This looks great.

    To fix the all page content should be contained by landmarks ACCESSIBILITY ISSUES change your <div class="container"> to <main class="container"> or wrap everything that's in the body with a <main>

    for the Level 1 heading error, use a level 1 heading instead of a h2 then style that heading to look the same as the h2

    otherwise this is really well made 👍 keep up the great work✨

    Marked as helpful
    0
  • Andrew•90
    @L1m1tzPosted over 2 years ago

    Hiii, I need improvements on my code so ama learn from yours. In my eyes this is a 10 out of 10 so Well done👏👏👏

    Marked as helpful
    1