Mick
@F9F6EEAll comments
- @Hasani-SediqaSubmitted 12 days ago@F9F6EEPosted 11 days ago
Hey! congrats to you for completing this challenge here are some feedback on your solution :
-
Try to reduce the size of the
<img>
so they have almost the same size has the heigh of theh3
and<p>
combined so it don't differ to much from the design. -
You forgot to add a
{border: solid}
on Patrick's picture (1-2px of border should be enough) with violet color. -
The
<svg>
is a bit too big, but have also the same problem and trying to figure out.
Except that, good job. Keep going !
Marked as helpful0 -
- @sportif7Submitted about 2 months agoWhat are you most proud of, and what would you do differently next time?
next time I would start with mobile design first and use flexbox.
What challenges did you encounter, and how did you overcome them?I had challenge time to figure out dimensions with Safari so I install Chrome browser.(It will not match design accurately because I didn't use figma for dimension).
@F9F6EEPosted about 1 month agoHi, you really did a great job, good structure and semantic. congrat and keep going !
0 - @iamporrrrrrrrrSubmitted 7 months agoWhat are you most proud of, and what would you do differently next time?
None
What challenges did you encounter, and how did you overcome them?None
What specific areas of your project would you like help with?None
@F9F6EEPosted 7 months agoHey ! 👋 great work on this challenge. Here are fews points to improve your design :
- Maybe you can increase the size of your
h1
or add a<br>
between "Eau" and "De" so it fit like the design. - add a
cursor: pointer
in the.cart span:hover{...}
I hope it help :)
Have a great day !
Marked as helpful1 - Maybe you can increase the size of your