Responsive Multi-Page Website

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 2 years ago

  Greetings, Eric Khang'ati! 👋

  Good work on this challenge! 👍

  As your solution report suggests, make sure to wrap all li elements in a ul or ol element and make sure that all direct children of ul or ol elements are li elements. For things like navigation links, you should put the link tags inside of the li elements, like this,

  <ul>
    <li><a href="/">Home</a></li>
  </ul>
  

  I'd also like to suggest taking another look at the responsiveness of your site, as not all of the content of the page can be seen because it gets cut off around 450px (before the layout becomes mobile friendly).

  And if an element takes the user to another page (for example, the "About us" link on the home page), there is no need to wrap it in a button element (just a link tag should do).

  Hope you find these suggestions helpful. 😊

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  0

  ---

  

  Eric Khang'ati•430

  @erickhangatiPosted almost 2 years ago

  @ApplePieGiraffe Hello there. Thank you for your feedback. Let me get into it and solve those issues.

  0