search

aliasmac / teacozy

0 stars 0 forks source link

Summary #5

Open farishkash opened 6 years ago

farishkash commented 6 years ago

Overall met expectations.

Using alt text is about accessibility for the visually impaired on the web. You used comments for organization which is great.

CSS needs to avoid repetition. If you are repeating the same styles find an alternative to reduce the amount of css you write. The less you write the less likely you will make an error.

Also to prepare you for the final projects, look at media queries. See if you can make this site fit inside of a mobile device by using one. You will be doing this heavily in your final projects.

https://developer.mozilla.org/en-US/docs/Web/CSS/Media_Queries/Using_media_queries

https://www.w3schools.com/css/css3_mediaqueries_ex.asp

aliasmac commented 6 years ago

Hi Farish, thanks for reviewing the project, I was quite behind with deadline so great have comments from you. Overall found the project quite difficult, but I'll try to incorporate your suggestions!