search

Lddr26 / tea_cozy

0 stars 0 forks source link

Overall Score #9

Open tommygebru opened 5 years ago

tommygebru commented 5 years ago

Overall Score

Criteria 1: HTML Structure
Score Level: 3 - HTML implementation enables proper use of flexbox.
HTML structure mostly reflects the grouping and flow of content in the web browser.
Comment(s):  Great stuff here Lisa, I want you to explore Semantic tags a little further, as well as local file paths for images!

Criteria 2: Visual Layout
Score Level: 4 - Layout matches the design spec in all sections 
(This includes a sticky header w/ no gap between it and top of viewport )
Comment(s):  👍 

Criteria 3: Use of Flexbox
Score Level: 4 - Flexbox properties are correctly applied to flex containers and flex items throughout. 
Flexbox is used properly in 5+ sections and is the primary way of positioning.
Comment(s): Good stuff here as well, I want you to get more comfortable with flexbox 

Criteria 4: Implementation of Design Specification and Content
Score Level: 4 - The page includes most styles and content provided in the design specification.
(For example, anchor colors match the design spec.)
Comment(s): 👍 

Overall Score: 15/16
tommygebru commented 5 years ago

Summary

Hey Lisa, well done on your project Teacozy, 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 Flexbox Froggy - A game for learning CSS flexbox and also A flexbox cheatsheet too 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 Lisa keep up the good work and happy coding!