css,html

Community feedback

• 

  raya•2,870

  @rayaattaPosted about 1 year ago

  Hello kotyba-km👋, congratulations on completing your first FED challenge 🎉

  I have some suggestions you might find useful.

  1 Try to make your html more Semantic by wrapping the main page content inside a <main> tag . Replace <div class="card"> With <main class="card">. This changes nothing visually but Using it makes all the difference. Using semantic markup improves

  SEO

  And user experience (accessibility) for people using assistive technology such as screen readers.

  2 The qr code should have text inside an alt attribute stating what it is and where it leads i.e alt="qr code leading to frontendmentor.io

  I hope this helps 🙃

  Otherwise your solution is neat👍

  Happy coding ✌️

  0