kurbart / tea-cozy Goto Github PK

https://github.com/kurbart/Tea-Cozy/blob/master/index.html#L1-L91

Make sure to comment the main parts of your HTML further in future projects. I noticed you had a few, but you can extend this further. Commenting becomes increasingly important as your projects become larger and more complex. Here is a quick refresher on commenting in HTML:

Summary Grade: Satisfactory

Good job! Your files are organized nicely, and you did a great job of matching the client specification for the most part -- this is an important skill to have as a developer and you seem to understand that. Moreover, you generally followed excellent code practice, so make sure to keep that up!

I have made some comments in your code to take note of. First off, make sure to comment the main parts of your code. You should do this for both your HTML and CSS files (especially your HTML). This becomes much more important as your projects get more complex and your codebase grows. Here is a quick refresher on commenting in HTML and CSS, respectively:

Your code is also very nicely structured/formatted and follows excellent code practices such as readability, indentation, etc. Overall, you did an excellent job.

This isn't a big issue now since you haven't learned about this yet, but I wanted to introduce you to the idea of responsive design. You will certainly learn more about this concept in the next project, but I wanted to provide a great resource to get you to start thinking about building sites that look good on all different kinds of platforms/browser sizes:

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

Great job overall! As a further challenge, try using css to design your webpage even further! You did a great job in following the client specification, but for your own practice you should see how creative you can get by taking the design/styling of your site to the next level (for example, you can try experimenting with background colors on your other divs).

https://github.com/kurbart/Tea-Cozy/blob/master/resources/css/style.css#L1-L113

Your CSS is short, clean and succinct -- make sure to keep it up. An area of improvement is to make sure to you are commenting your main styling. Here is a quick reminder on commenting in CSS:

In the next project you will learn more about responsive design, which is basically web design that lets you create content that fits different browser sizes/platforms. Here is a great resource to get you to start thinking about that:

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

https://github.com/kurbart/Tea-Cozy/blob/master/index.html#L2-L91

Your HTML is very nicely structured and you generally followed excellent code practices such as indentation, readability, commenting, etc. Make sure to keep up the great work!