All comments

• 

  Beto• 20

  @ZhyperMax

  Submitted 9 days ago

  

  Using flexbox for card

  1

  20

  

  moaz-shamim• 20

  @moaz-shamim

  Posted 1 day ago

  The solution does not contain semantic HTML. Inside the <main> tag, you are using too many <section> tags, which is not an optimal approach. Instead of using multiple sections, you should use the <article> tag for the card and include <header> or <footer> tags as needed.

  While the layout looks good, the card does not align properly across different screen sizes. Additionally, there is a noticeable difference in the design compared to your expectations. You can improve your design skills by practicing or by using a Figma file for proper style guidance regarding width, height, border radius, and padding.

  0
• 

  Daniel• 10

  @D4nielPSilva

  Submitted 15 days ago

  What are you most proud of, and what would you do differently next time?

  I changed the color of the texts, put the shadow on the qr code container and changed the cursor to default when placing the mouse over the container

  

  QR Code Component

  1

  20

  

  moaz-shamim• 20

  @moaz-shamim

  Posted 4 days ago

  Yes, the solution includes semantic HTML with appropriate tags for structure. It's accessible with readable fonts, proper contrast, and keyboard navigation, but could improve with more aria attributes. The layout looks good on different screen sizes due to responsive design with Flexbox or Grid. The code is well-structured, readable, and follows best practices, with only minor creative adjustments from the design.

  0