Skip to content
OverviewSolutions
5
Comments
5
teixeirabrenno
@teixeirabrenno

All comments

  • teixeirabrenno•70
    @teixeirabrennoPosted 5 months ago

    Don't worry if you didn't manage to get it right this time. Let's keep practicing and improving!

    Try to watch this video here: Responsive CSS Grid Tutorial - Angela Design (https://www.youtube.com/watch?v=68O6eOGAGqA) It really helped me to understand how to use CSS Grid and I'm sure you will understand it too.

    Marked as helpful
    0
  • Erik Norder•40
    @R3lssSubmitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    I'm really proud that i could organize everything.

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

    In the table part I didn't know how to get the same style as in the design file, but after a little bit off researching and thinking I obtained the result.

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

    How can I improvise that table in the "Nutrition" part

    Recipe Page Challenge

    1
    teixeirabrenno•70
    @teixeirabrennoPosted 5 months ago

    When I was doing this challenge I faced the same problem on the ''Nutrition'' part. I got it done by creating a table with 4 rows and putting a border below each row. It looks almost the same as the way you did but it's more practical and you can change the color of the thin line between each row if you want/need. I'm not sure if you can change the color of an <hr>. I tried and couldn't do it.

    I'll leave the link to my GitHub here so you can check it out how I did it, ok? https://github.com/teixeirabrenno/recipe-page

    Good job and let's keep improving!

    0
  • teixeirabrenno•70
    @teixeirabrennoPosted 5 months ago

    Good semantics and well structured code. Easily readable. I just want to point it out something in your CSS. You could have arranged the elements on the screen using just these lines here:

    .container{ display: flex; flex-direction: column; justify-content: center; align-items: center; }

    Good job and let's keep improving!

    0
  • teixeirabrenno•70
    @teixeirabrennoPosted 5 months ago

    Awesome job. It looks almost identical. One thing that I would like to point it out is that I think there are too much unnecessary divs in your code. I think that the code becomes a little bit easier to read using fewer divs. Other than that, it's great. Good job.

    2
  • teixeirabrenno•70
    @teixeirabrennoPosted 5 months ago

    I would say it's perfect. The only thing I would have done is to create a specific file for the CSS.

    0