All comments

• 

  Rone J Santos• 40

  @ronesanttos

  Submitted 10 days ago

  

  Parte responsiva usando o o display flex

  1

  10

  

  Michał Libera• 30

  @MlcodeDev

  Posted 10 days ago

  • Missing typography.
  • Missing the <main> tag.
  • Wrong units used for elements, you used px instead rem for text and em for padding and margin.
  • Try adding all the text into one container and set an max width so the text ends in a nice line.

  0
• 

  Rodrigo Alves• 30

  @rodrigoo-alves

  Submitted 10 days ago

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

  Im proud of the CSS because it took me no time to make and the html was easy to do and I have struggle with this two for a long time

  What challenges did you encounter, and how did you overcome them?

  I didnt encounter a challenge on this project

  What specific areas of your project would you like help with?

  If someone have a CSS tip or HTML that would improve my code I appreciate

  

  QR code card

  1

  20

  

  Michał Libera• 30

  @MlcodeDev

  Posted 10 days ago

  I love this project but you should use rem units rather than px for font-size so that it won't interfere with user preferences. Some people, especially old people like to set a larger font size, and rem is a scalable font size so it makes it possible to scale up, but when you set your font-size to px, it is unable to scale.

  Marked as helpful

  1

  This comment has received 3 repliesShow