search

AMA7103 / Tea_Cozy

0 stars 0 forks source link

Summary: Meets expectations #7

Open martoio opened 6 years ago

martoio commented 6 years ago

Good work! There's a few things you should take a look at. You follow the spec for the most part, but make sure you make your HTML a little cleaner. There's no need to wrap an h4 inside a div just to apply the style on the div. The less elements you use, the easier it will be for you to manage your code.

As a challenge, you can research and implement media queries to make your page responsive for different devices. You should be able to take a crack at this and make it good! 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/

Good job and keep up the good work!

AMA7103 commented 6 years ago

Thank you for the great and detailed review! I'll read the articles you have linked.