Skip to content
Submitted about 2 years ago

Responsive Sunny Side website using flexbox

Sandesh Adhikari•120
@sandeshad100
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

My 8th challenge here. For responsive devices I could t make the menu(nav) to hide/show properly please have a look at my code. I will appreciate your feedback.

Community feedback

  • Ahmed Kamel•1,190
    @hmadamkPosted about 2 years ago
    • well done a few notes though
    1. you must add an alt text to your image because screen reader will read that to blind people and if its not there it will read the src of the image which can be bothering for users, if you image is for decorating-perpose use an empty alt like so alt=""
    2. make sure that all of your page is contained by a landmark examples of landmarks are
    header for the top section
main for your main section
section with aria-label to describe why you added this section
footer for the bottom section
    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