Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive landing page using flexbox

@b-l-u-e

Desktop design screenshot for the Sunnyside agency landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


guys need help with JavaScript to toggle menu welcoming any thoughts and ideas am still learning java script

Community feedback

P
Chamu 12,970

@ChamuMutezva

Posted

  • <img src="images/logo.svg" alt="logo" class="logo"> , alt values should be descriptive to allow assistive tech users to visualize the message that is being put across. Values like logo are shallow and not beneficial as that will be automatically announced by the screen readers.
  • the following should be an unordered list of anchor elements
<div class="menu">
          <span>About</span>
          <span>Services</span>
          <span>Projects</span>
          <div class="contact">Contact</div>
        </div>
  • the best practice is to use one h1 element per page. Let your headings ascend in order.

Marked as helpful

1

Please log in to post a comment

Log in with GitHub
Discord logo

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