Qr Code Frontend Mentor

Community feedback

• 

  Autumn•110

  @AutumnCodPosted almost 3 years ago

  Hi!

  Well done to complete the challenge! There is a few suggestions.

  1. Accesbility: For the main content use the main tag. It will tell the people that everything inside the tag is the focus content of the page. And I would suggest to use the article-tag instead of section. -->https://webdesign.tutsplus.com/articles/the-truth-about-multiple-h1-tags-in-the-html5-era--webdesign-16824
  2. I am aware it is small project but I would always put my css in a seperate file and link them.

  Marked as helpful

  1

  ---

  

  Harshdeep Singh•220

  @JrDevHarshPosted almost 3 years ago

  @AutumnCod Thanks, I will remember in the future.

  0