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

All comments

  • Ayaat•460
    @Yazeed-gitPosted about 1 month 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 7 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 7 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 almost 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 over 3 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
  • Kevin Canlas•225
    @kvncnlsSubmitted almost 4 years ago

    Please critique my work! :D I'm trying to write cleaner code, so if anyone has any tips on being more efficient, I'm all ears!

    Pricing component with toggle

    2
    Ayaat•460
    @Yazeed-gitPosted almost 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
  • Nathaniel•50
    @NathanielCW87Submitted almost 4 years ago

    If anyone can see what I could've done to make it easier or faster, please feel free to leave a comment. Thank You.

    HTML, CSS

    1
    Ayaat•460
    @Yazeed-gitPosted almost 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 about 4 years ago

    cool solution man looks like the orignial 100%

    1