Simple web app

Community feedback

• 

  P

  Steven Stroud•11,890

  @StroudyPosted about 2 months ago

  Hey, Great job with this solution you should be proud, A few things I noticed,

  • Missing a <meta> description tag for SEO purposes,
  • It is best practice to have a <main> tag inside your body highlighting the main section.
  • There is no :hover effect when the mouse is over the links, easy fix CSS :hover Selector
  • ‍Using max-width: 100% or min-width: 100% is way more responsive then just width:100%, check out this article also from the same Frontend mentor dev responsive-meaning, she goes into more detail.
  • Using a naming convention like BEM, Using proper naming will prepare you for the changes in design of the website.
  • You should apply a full modern reset to make things easier as you build, check out this site for a Full modern reset

  I hope you found some of this information helpful, You should give the articles a good read and I look forward to seeing some more from you, Happy coding! 💻

  Marked as helpful

  0

  ---

  

  CyPhEr420•400

  @CyPhEr420Posted about 1 month ago

  Thanks man @Stroudy

  1

  

  P

  Steven Stroud•11,890

  @StroudyPosted about 1 month ago

  Hey @CyPhEr420, 22 days ago, My feedback has improved since then, Your welcome, You got this!

  Marked as helpful

  0
• 

  P

  Adam Whitehouse•130

  @adamwhitehouse95Posted about 2 months ago

  Nice! Close to the design, few things could be altered though

  Marked as helpful

  0