Simple page

Community feedback

• 

  polukarp•190

  @polukarpPosted about 2 years ago

  Love your solution, but I'd suggest a couple of minor fixes here:

  • Why is your <div className="nav-bottom"> not a part of your <nav> element but still a part of your Header component? Try placing it in your main content, especially as it contains the <h1> tag. You can name it a hero section if you wish.
  • Also your <ul> elements in the footer should not contain directly any element rather than <li>, or <template> elements.

  Marked as helpful

  0

  ---

  

  Bekzod•300

  @bekzodturgunoffPosted about 2 years ago

  @polukarp Thanks for commenting to my solution. I'll try to fix them.

  0