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

    @korniykom

    Posted

    • on page should be at least one main element.
    • to render list of buttons use ul and li
    • put name in h1 element
    • put quote in q element
    0
  • @Pablo-Zallio-Dev

    Submitted

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

    In this project I have become more organized and have better analyzed the figma design.

    P
    Max 70

    @korniykom

    Posted

    The first thing that catches the eye is the thickness of the border. It should be 1px. Even though there is no predefined class in Tailwind CSS with that thickness, you should've used border-[1px].

    Also, I think you should fix the height to 522px by using h-[522px].

    Your custom shadow has an offset of 10px in both directions, which is close to 8px but still not exactly 8px. Next time, take a closer look at the provided Figma file.

    Apart from that, your card seems identical to the reference. Keep up the good work!

    Marked as helpful

    0
  • @Kevinw14

    Submitted

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

    I don't think I got it close to the design but I am proud of something that does look visually appealing.

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

    Getting the height to width ratio correct. I used what I thought looked best.

    P
    Max 70

    @korniykom

    Posted

    There is a slight problem with the card ratio. You can find the correct one in the provided Figma file. You used w-80, which corresponds to 320px, which is the correct width, and h-[550px], but in the design, 499px was used (really interesting choice). So I suggest changing the height to h-[499px] or h-[500px]. Overall, the card looks very nice, though after you change the height, you may need to adjust the margins of the text as well.

    0