All comments

• 

  ildi0818• 40

  @ildi0818

  Submitted 6 months ago

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

  I think I'm starting to understand the essence of the mobile first and why we start coding with it first. Unfortunately, I didn't start with that, so it was more difficult. But it helped me understand.

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

  Dynamically scaling the image was a bit more difficult.

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

  The sizing of the image caused minor difficulties in order to reduce its size proportionately. I could not remove the bottom line of the table, I only set the same color as the background. I tried to use a css variable, but I wasn't sure if I should assign it to, for example, "li".

  

  My recipe page

  2

  30

  

  Robert Douglas• 90

  @robertdouglasaimon

  Posted 5 months ago

  Hello ildi0818! All good? I was looking at your project and it looked good!

  I saw that you assigned

  li { --color-li: hsl(24, 5%, 18%); }

  because I was unsure about assigning variables in CSS. Always try to assign variables to the " :root {} " property. Because, in addition to being designed for this purpose, it helps you keep the code clean and organized (And you can assign both the colors and the fonts that you import with @import in it! It helps a lot!)

  Ex of :root:

  @import url('https://fonts.googleapis.com/css2?family=Young+Serif&display=swap');

  :root { --font-Young-Serif:"Young Serif", serif; (for font) --cor1-Raspberry: hsl(332, 51%, 32%); (for color) }

  I hope I helped, your project turned out really good! Congratulations!

  Marked as helpful

  1
• 

  joeyturbo• 50

  @joeyturbo

  Submitted 5 months ago

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

  this was the first project without a figma file (i have a free account) and i think i managed to get everything sized up and proportional to the design file. i liked this project

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

  no big challenges.

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

  none

  

  social links profile

  1

  30

  

  Robert Douglas• 90

  @robertdouglasaimon

  Posted 5 months ago

  Honestamente, seu projeto ficou excelente! Única dica que eu daria agora era tentar acertar com o modelo (Digo altura, largura e etc..) e talvez evitar colocar sinais nas classes (_ , -, @, etc..), no mais, ta muito bom.

  Bom trabalho! :)

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Pedro Potense• 50

  @PedroP16

  Submitted 6 months ago

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

  Estoy feliz por haber podido realizarlo sin ninguna complicacion:

  • De haber usado flex para centrar la card
  • De poder manipular la imagen para que no se salga del contenedor
  • Haber implementado los colores como variables

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

  Personalmente no encontre ninguna dificultad, no es el primer Challenge que hago (Puede ser que haya errores y no lo sepa).

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

  Me gustaria recibir ayuda en cualquier parte del codigo si se cree que se podria haber hecho de una mejor forma.

  

  Responsive Card Using Flex

  1

  30

  

  Robert Douglas• 90

  @robertdouglasaimon

  Posted 6 months ago

  Great! :D

  Now just adjust the color and size of the text and the size of the fonts.

  A good job.

  0
• 

  Sumaiya92• 20

  @Sumaiya92

  Submitted 6 months ago

  

  CodeTag: HTML/CSS QR Creator

  1

  20

  

  Robert Douglas• 90

  @robertdouglasaimon

  Posted 6 months ago

  Hello budy, how are you? Some tips that can help you! It would be nice if you separated the CSS from the HTML to keep the code clean. It would help you understand where you need to improve in the code.

  In the 'body', the 100vh min-height is enough, you just need to adjust the display to flex, align the items in the middle with align-items: center. And here in the body itself, apply the font-size to the entire body as specified by the challenge: 15px.

  You named a div in the style, but didn't specify which div it is. And applied several properties to this call, which was for almost all div's (There was a possible conflict). Due to the wrong DIV entry, it would be a good idea to adjust the div entry and the parameter indentation.

  0