QR Code component using Flexbox and Unsemantic grid framework

Community feedback

• Account deleted

  Hi, Good job 👍

  Your design looks perfect 😃. Please also increase the border-radius to make it perfect ✔

  Good luck 🍀

  Marked as helpful

  0
• 

  JoshuaWBlick•270

  @JoshuaBlick94Posted almost 3 years ago

  Thank you, let me implement changes straight away

  1
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! Good Job you made it look nearly perfect to the preview

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body like it should be your container

  -> For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc

  -> But use header tag only once in main heading element.

  Keep up the good work!

  0
• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted almost 3 years ago

  Consider using semantic elements such as the main element as stated in the automated feedback. The main element is considered important as that is where the main content of the site should reside. Alt values should be descriptive for the benefit of assistive technology users.

  0
• 

  jemeneradev•580

  @jemeneradevPosted almost 3 years ago

  I would round up the corners a bit more, maybe 15px or bigger. The design calls for rounder corners. Also, you might consider reducing the "line-height" property of the paragraph. Good job!

  0