WXSanchez
@WXSanchezAll comments
- @Mohamed-OdaySubmitted 4 months ago
- @joshhovisSubmitted 4 months ago@WXSanchezPosted 4 months ago
I think you just forgot to add the fonts in. But other than that looks great!
1 - @jdgarcia277Submitted 4 months agoWhat 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.
@WXSanchezPosted 4 months agoLooks great! I think you missed the bold font on the prep time section but great job!
0 - @deyorzSubmitted 5 months ago@WXSanchezPosted 5 months ago
Looks good. I did the same thing where I removed the attribution. If you had it, it probably would have been perfect.
1 - @joaotfrodriguesSubmitted 5 months agoWhat 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.
@WXSanchezPosted 5 months agoThis 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 helpful1