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

All comments

  • Ayaat•460
    @Yazeed-gitPosted 6 months ago

    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.👍👍

    0
  • Ayaat•460
    @Yazeed-gitPosted 12 months ago

    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.

    1
  • Ayaat•460
    @Yazeed-gitPosted 12 months ago

    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
    0
  • Ayaat•460
    @Yazeed-gitPosted about 3 years ago

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

    Marked as helpful
    0
  • Ayaat•460
    @Yazeed-gitPosted about 4 years ago

    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

    0
  • Ayaat•460
    @Yazeed-gitPosted over 4 years ago

    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!

    1
  • Ayaat•460
    @Yazeed-gitPosted over 4 years ago

    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

    0
  • Ayaat•460
    @Yazeed-gitPosted over 4 years ago

    cool solution man looks like the orignial 100%

    1