QR code component using flexbox

Community feedback

• 

  P

  Margaux•200

  @margaux-works

  Posted 11 months ago

  Hello!

  good job in posting your first solution :)

  What I noticed:

  • you correctly implemented the border radius for the card, but a border radius is missing on your image (see original design provided)
  • in order to use semantic html, you could use <main> instead of a simple <div> for the card
  • for your text, consider using headings such as <h1> for your main title, so that you are structuring your content effectively.

  The last 2 comments would not optimised how your page looks but would be helpful for SEO and accessibility purposes.

  Hope this helps!

  01

  Marked as helpful
• 

  Farid•30

  @Faridius

  Posted 11 months ago

  The solution effectively includes semantic HTML, enhancing both accessibility and SEO. However, it can be improved by adding meta tags for better accessibility and search engine optimization. The layout is responsive and looks good on a variety of screen sizes. The code is well structured, readable, and reusable, and follows best practices. Additionally, the solution closely follows the original design.

  01

  Marked as helpful