Qr code component using CSS flexbox

Community feedback

• 

  Tamir-Nahmias•60

  @Tamir-NahmiasPosted about 2 months ago

  overall , your website looks great! very similar to the original design we were given . nonetheless ,in your next project, consider adding semantic HTML elemetns inside the body tag such as "main" and "footer" (for the card content , and the "Attribution" section correspondigly), they are excellent for accesibilty and for SEO ranking. also, for non semantic elements such as div, try to add roles . the layout of the screen is just fantastic, and corresponing very well to diffrent sizes , altough I didn't see min size for mobile phones. your code is very well written , you succeded your task with minimum div elements and brought fantastic results! (btw, I've learnt a thing or two from your desgin , thanks!)

  Marked as helpful

  1
• 

  dharul619•150

  @dharul619Posted about 2 months ago

  menyalaaa

  1