ansteadm / prj-rev-bwfs-dasmoto Goto Github PK

Check out this article that explains why you need to care about using special characters in HTML: http://text-symbols.com/html/encode/
The ampersand should be written as:
&

Good job on separating the HTML into sections! One thing you might want to consider is changing the <div> into a <section> to improve the semantics of your code.
If you aren't familiar with semantic HTML: http://www.hongkiat.com/blog/html-5-semantics/
Example:

<!-- First section -->
<section> <!-- or <div>, but <section> is more semantic -->
 <h2 class="brushes">Brushes</h2>
 <img src="#" alt="#" />
 <h3>Text</h3>
 <p>
  Lorem <span class="price">ipsum</span>
 </p>
</section>

You can put the font-family on the html or body elements, that way you don't have to constantly specify the font for all the elements:

Don't use absolute paths. I don't have this path on my computer, so it won't work when I download your project. Use relative file paths.

https://www.kirupa.com/html5/all_about_file_paths.htm

Good job on the code! Your code meets the spec and is free from any major errors (just make sure you get the file paths correct). Take a look at the previous issues to polish up the code and make it more professional.

In terms of improvements, I would suggest taking a look at the article about semantic HTML and trying to improve your HTML to make it more semantic (ie, meaningful). It seems like an irrelevant thing, but it's what makes your code stand out as professional.

If you are interested in learning more about some HTML/CSS best practices, check out some of the links below:
https://github.com/bendc/frontend-guidelines
http://learn.shayhowe.com/html-css/writing-your-best-code/

Good job on this project and keep up the good work!