QR code component

Community feedback

• 

  Rahul K Dinesh•290

  @b16h22Posted 2 months ago

  Saeedakrami's solution of the QR Code component looks identical to the design file. Code is readable and generally good. I felt some of the code could be cut down in size like using <img> tag instead of the <div> tags for the QR Code component.

  0
• 

  ManPP•130

  @WhitezerDPosted 2 months ago

  You should use more semantic tags than non-semantic ones like div tag.

  1. Use h1 for the title of the pages, only one h1 tag per page
  2. Use p tag instead of div on your class text-paragraph
  3. Delete unnecessary tag that doesn't matter ex. div on line 37

  0