Parvej Shah• 10
@parvej-shah
Submitted
Ayaat
@Yazeed-git
All comments
- Ayaat• 460
@Yazeed-git
Posted
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 - yamuna001• 140
@yamuna001
Submitted
Ayaat• 460@Yazeed-git
Posted
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 @ErickDev00
Submitted
Ayaat• 460@Yazeed-git
Posted
Good work, Cool load animation, The picture doesn't load so you may have forgot to include it
Marked as helpful
0- Paras Thapar• 40
@paras2707
Submitted
How can I improve responsiveness of this webpage
Ayaat• 460@Yazeed-git
Posted
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
@kvncnls
Submitted
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!
Ayaat• 460@Yazeed-git
Posted
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
@NathanielCW87
Submitted
If anyone can see what I could've done to make it easier or faster, please feel free to leave a comment. Thank You.
Ayaat• 460@Yazeed-git
Posted
Nice work with the card matching the design, although i think you should work on:
-
making the shadow blur lighter
-
design fonts aren't coming through, so link ur code to the used google font
-
white space/padding is needed at the bottom section of the card
0 -
