drood87 / dasmotoartsncraft Goto Github PK

Overall met expectations

In this first project I look at two primary items with the main goal being to stop inefficient or bad practices early.

First is did you use divs, ids/classes to manage and organize your html content.

Great on the classes/ids. Div usage needed some work. This is a point of emphasis as you expand into flexbox and styling in general were div is the center piece.

Second is your css efficient and non repetitive.
You did great here and avoid the common mistakes that I usually see in this first project.

One other great thing you did is you used comments to mark each section in your html.

To prepare you for the upcoming projects I like you to go to flexboxfroggy.com. See how far you can get, the later projects uses flexbox heavily and this will help you immensely.

So the only thing on this site is we are looking for each section to be put into a div. So basically the brushes, frames, and paint section from h2 to the paragraph.

This places each one in a distinct container element.

<h2 class="fonts brushes">Brushes</h2>
	<img src="./resources/images/hacksaw.jpeg" alt="Picture of Hacksaw Brushes">
	<h3>Hacksaw Brushes</h3>
	<p>Made of the highest quality oak, Hacksaw brushes are known for their weight and ability to hold paint in large amounts. Available in different sizes. <span class="pricing">Starting at $3.00 / brush.</span></p>