Skip to content
OverviewSolutions
5
Comments
5
joeyturbo
@joeyturbo

All comments

  • Samuel Sylvester•330
    @Sylvester009Submitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    The time taken to complete the task, which is less than usual, shows that I am getting the hang of it.

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

    still the centering of my main content.

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

    centering

    Responsive Product Card

    1
    joeyturbo•50
    @joeyturboPosted 7 months ago

    Hey this looks great. A few things I would change:

    • the Fraunces font didn't appear to make it over.
    • limit the height of the card to accommodate space between text in desktop view
    • add a little more padding to the text in desktop view

    great work!

    0
  • joeyturbo•50
    @joeyturboPosted 7 months ago

    Looks pretty good. Here are some things I would update:

    • fonts, font colors and background colors should match the style guide
    • use text indent to fix indent issues with list items that exceed one line
    • use grid layout on the nutrition details to keep everything aligned properly
    • use media queries to replicate the mobile design provided in the project
    0
  • joeyturbo•50
    @joeyturboPosted 7 months ago

    looks nice. a couple things i would change:

    • it doesnt look like the Inter font made it over.
    • the black text at the top of the profile should be white.
    • some of the font sizes and weights need to be adjusted to match the design file.

    good work :)

    0
  • joeyturbo•50
    @joeyturboPosted 7 months ago

    Looks good. Some things I would change:

    • hyperlink the title and set hover color to yellow
    • add a little more padding to the main container
    • apply the project font to the body element (the author name at the bottom is in times font)
    0
  • joeyturbo•50
    @joeyturboPosted 7 months ago

    this looks great. i think the container might be a little bit bigger than the one in the figma file but otherwise looks good. great job!

    0