True newbie solution. Your feedback are welcome :)

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted over 2 years ago

  Hello Tales, congratulations for you solution, you did a good work!

  I saw your live site and everything is fine, there's only two things you can consider changing to give it a final touch!

  1. The backgroud-color is applied to the container, this means that the background will respect the container css properties, in this case your background isn't filling the whole screen, to fix that all you need to do is apply the background to the body.

  body { background-color: var(--cream);}

  2. Padding-top, your solution seems great, but the padding between the text elements need to be a little bit bigger to match the design file, I saw that you've applied justify-content: space-between; but if you check the design file you can note that the paddings are not the same for all elements, although was a really smart choice! You've could reach the same result using the property gap.

  Congratulations Tales, you've a really good solution! Keep coding

  Marked as helpful

  1