search

dmitch03 / TeaCozy

Tea Cozy project for review
0 stars 0 forks source link

SUMMARY #5

Open aubreywullschleger opened 7 years ago

aubreywullschleger commented 7 years ago

Grade: Meets expectations

Summary: Nice work on this project! I love your organized, semantic HTML, your use of locally stored images, and your use of flex-box in CSS.

A couple things to work on here would be: 1. using descriptive alt attributes on your <img> elements and 2. using group selectors to make your CSS more concise.

Next, as a stretch goal, I challenge you to research and implement media queries to make your page totally responsive for all screen widths. This is a challenge I think you are ready for based on your work with this project! Here are a couple resources to get you started: https://developer.mozilla.org/en-US/docs/Web/CSS/Media_Queries/Using_media_queries and https://css-tricks.com/snippets/css/media-queries-for-standard-devices/

dmitch03 commented 7 years ago

Thank you very much! Flex-box was definitely the most challenging thing for me so far in the course. Glad to hear it looked decent. Also, I made a real effort after the first project review to comment and organize my code a little more. I think it's pretty satisfying to see the clean, organized result. I've been working on unit 5 this week and learning about media queries. I'll give them a try on this!