Mohamed-Oday• 190
@Mohamed-Oday
Submitted
WXSanchez
@WXSanchez
All comments
- WXSanchez• 80
@WXSanchez
Posted
Looks great other than some sizing and spacing.
0 - PJoshua Hovis• 200
@joshhovis
Submitted
WXSanchez• 80@WXSanchez
Posted
I think you just forgot to add the fonts in. But other than that looks great!
1 What are you most proud of, and what would you do differently next time?
Happy to complete the last intro design!
What challenges did you encounter, and how did you overcome them?
Originally I applied padding to my article which caused padding around the image for the mobile view, which didn't fit the design. I quickly adjusted this by adding space to the individual sections themselves.
What specific areas of your project would you like help with?
I just want to make sure I used the correct semantic html.
WXSanchez• 80@WXSanchez
Posted
Looks great! I think you missed the bold font on the prep time section but great job!
0- João Rodrigues• 70
@joaotfrodrigues
Submitted
What are you most proud of, and what would you do differently next time?
I began improving the website's accessibility by implementing several changes, such as using CSS preload to speed up loading times, replacing pixels (px) with relative units (rem), and substituting divs with semantic HTML elements.
What specific areas of your project would you like help with?
I would like assistance with correcting the semantic HTML in my project and identifying any missing best practices.
WXSanchez• 80@WXSanchez
Posted
This looks great. I think you may be missing a bit of box shadow at the bottom (hard to see and I'm not confident it is actually there) but your use of best practice accessibility and speed components is great.
Marked as helpful
1
