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

  • @geraldesteban

    Submitted

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

    Build using react and arranged components in different files.

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

    In the nutrition part where I put lines using react

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

    Maybe clear structuring if this component is a bit confusing.

    angwb 40

    @angwb

    Posted

    I like that you keep the background color from the solution. It looks pretty and I haven't dwelled that much in React , hopefully soon. I like the naming of the classes you implemented as they are easy to place in the design. In the responsive view the table would be clearer if the info is not that close. Perhaps a margin would be useful there.

    0
  • angwb 40

    @angwb

    Posted

    I think your solition includes semantic HTML as I can see the tags used in your index. The layout looks good and clear in a smaller screen size although I can't see the hover effect for the links. I think all in all it looks similar to the suggested solution. Perhaps the text from the bio could be centered in order to look more similar to it, but I don't see major changes are needed.

    0
  • angwb 40

    @angwb

    Posted

    I see that you included sematic html. It seems adapted to a different screen size without problem. I would add the suggested fonts for this project so that it looks more cohesive overall. The code is readable and well-structured but I would use a more general name for the classes e.g instead of "learn" - "card-text" or something related. I don't think the solution differs much from the solution. Congrats.

    0
  • angwb 40

    @angwb

    Posted

    I think the layout looks good according to what was asked in the challenge I don't think it differs from the design. The code seems well-structured and readable. I like how you work with flexbox, I think it's a good choice since this challenge doesn't require many elements across the axis. I like how you worked with Flex only on body, I did not do that and it might explain some things on my solution.

    0