z-mn• 150
@z-mn
Submitted
What specific areas of your project would you like help with?
Any form of feedback as to where I can improve would be appreciated.
tris6
@tris6
All comments
- tris6• 70
@tris6
Posted
Great code organization and nice use of reusable skin classes! And thanks for introducing me to place-content property. (Where this been all my life? I've been using two lines like a sucker!)
Just a little flag that your H1 is a little smaller, thinner and spacier than the design. But, I do prefer your look to the mockup, so not neccessarily a bad thing.
Marked as helpful
0 - Diego Alves• 190
@DiegoAlvees
Submitted
tris6• 70@tris6
Posted
Nice work! I can't work out why ingredients and nutrition section have the left padding, couldn't see a code issue causing it. For the >tablet media query it looks like <H1> design has more letter spacing. I missed this as well, only noticed after submitting.
0 - yeswanthsaipyla• 10
@yeswanthsaipyla
Submitted
tris6• 70@tris6
Posted
Nice! Great idea to use <strong> semantics for "Learning".
0 - Olaniyan Kafayah• 280
@olaniyan18
Submitted
tris6• 70@tris6
Posted
Looks good! Great use of React components. CSS structure easy to follow.
0 - GOPALA-KRISHNA-1• 20
@GOPALA-KRISHNA-1
Submitted
What are you most proud of, and what would you do differently next time?
I DID WELL..I THINK
tris6• 70@tris6
Posted
Looks great! Love the attribution. I think 250px img width would equal the design closer, and makes text read easier. But that's a nit-pick.
0
