Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • P
    Yi-Ting Lu 100

    @kris-lu-dev

    Posted

    Hi Gabriela,

    I've just finished this challenge as well. I'd recommend you start to learn a bit knowledge about responsive image. Let's keep up with the challenges 💪🏻

    Marked as helpful

    0
  • P
    Yi-Ting Lu 100

    @kris-lu-dev

    Posted

    Hi Moustafa,

    Great job! I like how you've organized the typography variables.

    I did notice that the nutrition section is missing from the solution, though. This would be a great chance to practice styling HTML tables. It would also be beneficial to add a mobile-responsive version.

    I hope you find my feedback helpful! Happy coding! 🚀

    0
  • P
    Yi-Ting Lu 100

    @kris-lu-dev

    Posted

    Hi @aymenezz,

    You did a good job!

    But I've noticed a few points you can try to make a improvement:

    1. Centerlize the profile:
    • Set the height of body to 100vh
    • Remove the height of .profile
    1. Apply font-family as Inter and set
    @import url('https://fonts.googleapis.com/css2?family=Inter:ital,opsz,wght@0,14..32,100..900;1,14..32,100..900&display=swap');
    
    body {
    font-family: "Inter", sans-serif;
    }
    

    Keep going!

    0
  • P
    Yi-Ting Lu 100

    @kris-lu-dev

    Posted

    Great job! However, downloading the Figma file to adhere to the design system and polish the details would definitely improve it!

    Marked as helpful

    1
  • P
    Yi-Ting Lu 100

    @kris-lu-dev

    Posted

    Looks perfect!

    0