QR Component Main - Javier L. Pinto

Community feedback

• 

  Gurbala Laurent•90

  @laurentGurbalaPosted about 2 months ago

  Great job on your project! Regarding the HTML semantics, I think you could change the div tag that wraps your entire project to a main tag. As for the CSS, be careful with the minimal phone screen size, as your card shrinks too much. It might be better to keep the same dimensions regardless of the screen size. Overall, well done, it's a beautiful project!

  Marked as helpful

  1

  ---

  

  Javier Leonardo Pinto•20

  @JavierPintoVasPosted about 2 months ago

  @laurentGurbala Thank you very much for your comments! I would like you to tell me more specifically because I am learning and I really don't know how I would make those changes you mention because even the image doesn't look quite right when changing to cell phone.

  0