aizanAhmed / Colmar_Academy

1 stars 0 forks source link

Overall Score #8

Open tommygebru opened 5 years ago

tommygebru commented 5 years ago

Overall Score

Criteria 1: HTML Structure
Score Level: 4 - HTML implementation enables proper use of media queries.
HTML structure reflects the grouping and flow of content in the web browser.

Criteria 2: Visual Layout for Both Desktop and Mobile Sizes
Score Level: 4 - Desktop and mobile layouts match the wireframe in all sections. 

Criteria 3: Responsive Design (Media Queries, Responsive Units, etc.)
Score Level: 4 - Media queries kick in at appropriate breakpoints. 
Responsive units are used throughout.

Criteria 4: Visual Design and Accessibility (Color Palette, Typography, Transitions, etc.)
Score Level: 4 - An accessible color palette was chosen. Typographic considerations were made. CSS transitions were used.

Overall Score: 16/16
tommygebru commented 5 years ago

Summary

Hey Faizan, well done on your project Colmar, it's clear you have an excellent understanding of HTML and CSS basics. Spend a little more time reviewing/practicing upon the feedback until it clicks, and I'm confident you'll do fine.

Also keep some HTML/CSS best practices in mind when coding A great way to review content on the Go and also Become familiar with some of your Browser's web tools When in doubt Google it (or MDN it) 🎉

tommygebru commented 5 years ago

Next Steps

You can update your project to include any of the suggestions I addressed today. Also if you choose to you can simply upload a new github repo, updated to reflect any feedback 👍

Ok Faizan keep up the good work and happy coding!

aizanAhmed commented 5 years ago

Hey @tommygebru Thanks very much for all your kind words. I am feeling very confident after reading your review. I have updated the project. Font awesome is linked too. Your suggestions are also taken care of. Feel free to look into it.