HTML5, CSS3, FlexBox,

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Greetings, King David! 👋

  Congratulations on completing your first Frontend Mentor challenge! 🎉 Nice effort on this challenge! 👏

  A few things I suggest are,

  • Making sure any <li> tags that you use are direct children of a list element (such as <ul> or <ol>). That will clear up those errors on your solution report.
  • Keeping the desktop layout for a little longer (and not switching to a mobile layout immediately at 1440px) since there is still space the desktop layout for a while (and that way, the design will fill up a little more space on smaller desktop screens).
  • Adding a favicon to your site (there's one that comes with the free starter files that you can use).

  Keep coding (and happy coding, too)! 😁

  0

  ---

  

  King David•20

  @ronalking182Posted over 3 years ago

  @ApplePieGiraffe Thanks for ur comment, I really appreciate, it means a lot . I will try my best to work on everything u pointed out One last question pls -how do I use the favicon(where in my html should I add it )

  0