HTML, CSS, Flexbox

Community feedback

• 

  Faris P•2,810

  @FarisPalayiPosted over 3 years ago

  Looks good and responds well 👍

  Some of my suggestions are:

  • In the mobile version of the site, there's a horizontal scrollbar. You can remove that by using the overflow-x property, or you can find which element is causing the overflow and make appropriate changes.
  • Add :hover states for interactive elements like buttons. (You can also add :focus and :active state changes if you want to)
  • Also, add cursor: pointer to interactive elements to better indicate interactivity.
  • The text and social media icons in the footer and the text in the <nav> element should be links, since most of the time those will be a link to another webpage.
  • You are using <p> tag for headings. You should use a <h1> -<h6> tags for that. Learn about Semantic HTML5 Elements
  • In addition to that, clear those issues in the reports section as well.

  I know it's a lot. Take your time to understand these things. No need to rush. And Have fun coding ✨

  1

  ---

  

  Obieze Ezeugo Felistus•130

  @FelistusPosted over 3 years ago

  @FarisPalayi Thank you Faris P for the response. I will look into that. But with regard to the overflow, I have tried to locate the div or section causing it but to no avail. Used colours to help me locate the unresponsive part but can't find none..

  0