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
    Daniel 70

    @DAJ350

    Submitted

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

    I'm proud of how closely I was able to recreate the design using based of the provided Figma design file.

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

    I had a slight challenge figuring out how to achieve the tablet and mobile dimensions but was able to overcome this using a couple media queries.

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

    I'm open to suggestions.

    @TheTrueScout

    Posted

    Someone suggested this to me, and I think it'll help you too :) it'll make your card more responsive.

    .card {
    width: 90%;
    max-width: 37.5rem;
    }
    
    0
  • @TheTrueScout

    Posted

    It looks like you set a height on the container. Sometime's that's bad practice because you limit things unnecessarily. Remove the height, and add more padding to the bottom and you'll be a'ok.

    I just looked at the live site, and it looks like that has already been fixed- which is good. ( the code still has a fixed height, but if it works, it works :). )

    Learn how to use flex boxes. It will make your life much easier.

    You can use margin: 15px 0 15px 0; in order of Top, Right, Bottom, Left instead of using margin left, or margin top on separate lines.

    Styling the image is unnecessary. adding a border radius is enough.

    You didn't add the designated challenge font.

    0