Skip to content
Submitted almost 3 years ago

sunnyside-agency-landing-page-main

Rahib K•350
@rahiii-dev
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Agency landing page
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

first time with big webpage :)

Community feedback

  • Anosha Ahmed•9,300
    @anoshaahmedPosted almost 3 years ago

    To get rid of the accessibility/HTML issues shown in your Report:

    • wrap everything in your body in <main> ... OR use semantic tags ... OR give role="" to the direct children of your <body> ... Click here to read more
    • start your headings with <h1>, and move up by one level each time
    • <section> and <article> usually need a heading; so if you don't need a heading in it, use some other element such as <div>
    • <a> should have an aria-label ... Click here to read more
    • use classes instead of id's, if you're going to use the same one more than once

    Great job! :)

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord