@kodan96 Unfortunately, **this needs a lot of work**. - Why did you make your `class="header_menutoggle"` a `div`...❓This is meant to be an interactive element and a `div` is **definitely not**. Instead this **should be** a `button` and it should have an `aria-label`, `aria-expanded` and `aria-control`. * This is not necessary, instead use JS to change the `img` of the `button` when the `aria-expanded` changes. ```

``` * Lastly, the `button` should be **inside** the `nav` and it should be the `first` thing inside of it. - Whenever you remove the default styling for `list` elements, you are **removing** their semantic meaning. **To correct this**, you need to apply the `role="list"` to the `ul`/`ol` and the `role="listitem"` to the `li`. * For the logo, **remove** `

`, it is **unnecessary**. Once that is fixed, the `anchor` element **should have** an `aria-label` stating only the sites name and and where it leads to; so in this case it would state "Scoot - Home". - All the background patterns, **should be applied via CSS** and not be present in the HTML. All you are doing it bloating your code. * The `

` should have a `visually-hidden` heading and the `

` should **be built** using the `list` element. - This `

Locate with app

`, `

Pick your scooter

` and `

Enjoy your ride

` should then be `h3` headings. * The Apple and Google store should **each be wrapped** in an `anchor` element so users can then be directed to the appropriate store to download said app. - The `footer` logo should be **wrapped inside** and `anchor` element. * This `

バレンタイン 😈

All solutions

Loopstudios [Landing Page]

Photosnap multi-page website

Scoot Website

Art Gallery Website

Interactive Rating Component 👻