bombardier200 / prj-rev-bwfs-tea-cozy Goto Github PK

https://github.com/bombardier200/prj-rev-bwfs-tea-cozy/blob/master/Project%202/index.html#L13

Seems this one doesn't actually work to take me anywhere on the page, i'd check the spelling of your ids.

https://github.com/bombardier200/prj-rev-bwfs-tea-cozy/blob/master/Project%202/index.html#L25-L26

Some spacing here could help separate things logically.

https://github.com/bombardier200/prj-rev-bwfs-tea-cozy/blob/master/Project%202/CSS/style.css#L1

Spacing, comments and indentation can really help make your CSS much easier to read, check out this style guide. https://google.github.io/styleguide/htmlcssguide.html

Grade: Exceeds Expectations

Summary:
Well done here, your code is both functional and well organized, meeting our spec and easy to read. Both functionality and readability are about equally crucial in the programming world, and you seem to have paid sufficient attention to both, so well done. I would stress you try using some more spacing in both HTML and CSS, along with showing some commenting love to CSS as well.

For future projects, try throwing in your own little bits of style to dress up the site and personalize it. It's good to practice design along with functional code, so use these templates as opportunity to try some things. http://www.webdesign-inspiration.com/web-designs/style/modern

https://github.com/bombardier200/prj-rev-bwfs-tea-cozy/blob/master/Project%202/index.html#L26

Great work using the flex-box for positioning, elements look good and are set where we want them.

https://github.com/bombardier200/prj-rev-bwfs-tea-cozy/blob/master/Project%202/index.html#L17

Nice organizational comments, these really help in readability for both me and you.