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

  • @Dribbz

    Posted

    Hello Great work, i would add set max-width to overall card and center it in the middle of the page rather than having it filling the entire page which isn't very esthetic to look at , overall you did a great job !!

    have fun coding !

    0
  • @Dribbz

    Posted

    Great animation tho i didn't see it at first !

    1
  • @Dribbz

    Posted

    Hallo , I would suggest you to try to immitate the design provided by frontend mentor so we cant critique your work more appropriately. Goodluck friend !

    1
  • Dipto Das 70

    @Das-Dipto

    Submitted

    I have just made this project using ReactJS and Tailwind CSS also with Vanilla CSS media query . Now I am expecting appropriate feedback from the expert of this community.

    Could anyone please inform me, are there any flaws in my project or not ?

    Thanks in advance.

    @Dribbz

    Posted

    I'm no expert but I will give you my input on your solution,

    Great job! the design looks fairly close I'd just do the following :

    • Center the container in the middle of the page Using "flex box"

    *Border radius on the button

    *"Perfume" header should have small font-weight

    *The Design has some issues scaling down

    • Use Media query for changing the img on scaling down -Use Min-width property to prevent the overflow that happens on the 150px width

    Goodluck !

    0