Skip to content
OverviewSolutions
29
Comments
8
Ayaat
@Yazeed-git

All comments

  • Ayaat•460
    @Yazeed-git

    Hey @tloyan,

    Great job, At first when I saw typescript and framer motion I was wondering what you needed them for cos of how simple the design is, when I saw the animation, I got it.👍👍

  • Ayaat•460
    @Yazeed-git

    Looks good @Parvej, I like the even padding all around the box. If you want yours to look more like the design, you need more padding on the bottom of the card, after the last piece of text, otherwise good job.

  • Ayaat•460
    @Yazeed-git

    Your solution looks great 👌, and there's a couple of lines I could learn from your code.

    For feedback, the only element to adjust would be the title(h1); 'Gabrielle Essence Eau De Parfum', the line-height should be less as the text is big and bold and requires less vertical spacing. Cheers

    Marked as helpful
  • Ayaat•460
    @Yazeed-git

    Good work, Cool load animation, The picture doesn't load so you may have forgot to include it

    Marked as helpful
  • Ayaat•460
    @Yazeed-git

    The site looks identical good job, the scaling of the web elements like the logo and the hero image could be reduced a little bit, especially around 768px - 1080px. Other than that, looks good

  • Ayaat•460
    @Yazeed-git

    Nice html structure and overall clean code, I think the transition on the primary card button is cool the one on the secondary_card's button though, i think, can be improved, or made similar to the primary. Overall Great Job!

  • Ayaat•460
    @Yazeed-git

    Nice work with the card matching the design, although i think you should work on:

    1. making the shadow blur lighter

    2. design fonts aren't coming through, so link ur code to the used google font

    3. white space/padding is needed at the bottom section of the card

  • Ayaat•460
    @Yazeed-git

    cool solution man looks like the orignial 100%