After days and days and days of trying to finish this project, I finally did it. To be honest, I almost cried hahaha I read so many of the posts here and didn’t feel lonely with the struggle. Here is my code in case anyone is interested in reviewing it. I would love to receive some feedback since this is my first post and been having a bit of impostor syndrome feeling this is not for me. Thanks to all who get to read my message. Hugs!
Great work finishing this project! The Tea Cozy can be a doozy!
Some things I noticed:
We want to make sure that we’re using semantic HTML wherever it’s applicable. Instead of
<div class="header"> the
<header> element would be the correct tag to use.
When creating lists, the direct child is always
<li> meant as list-item.
<ol> is the other type of list, meaning ordered list.
It’s much easier to work with relative pathing. From the looks of the directory, relative pathing would look like
Lastly for the HTML, I noticed none of the images have
alt tags. We want to ensure that images are accessible to all users, and unless they’re decorative images or other related non-contextual images, need to include
alt tags at all times.
As for the CSS, there were some small things I noticed. However, they did not negatively impact the design as it looks similar to the spec. As you complete more projects, you should also start to see where CSS can be redundant and learn more efficient ways of achieving the design you’re looking for.
Keep up the great work!
Thank you so much!! I first wrote the list of the nav bar with
- but I guess that’s not the correct one to use hehe I’ll keep practicing!