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

Your indentation is off in your HTML. Try highlighting your code, right clicking and selecting format. If your text edit doesn’t have that feature try install a prettifier plugin. Also, most production standards is 2 spaces.

Your HTML looks really good. You use all the right elements. Another thing is that you wrap your code segments with divs. One suggestion is to make sure you wrap your h1 on line 9 with a div too. Also, even though it’s a very small code base, sprinkle in some comments as it is a good habit to get into.

Overall you did a good job on your project. You used the right HTML elements and you wrapped most of your code segments with divs. Make sure to get your indentation fixed and try to add some comments. Your CSS was really good as well. Just make sure you set font-family only once. All in all, nice work.

Your CSS is on point. You use all the right classes and your end product looks a lot like our solution. One thing I noticed is that you assign font-family to a lot of elements. Try just setting it to body and all of your child elements will inherit it. That way you set it only once. Lastly, when you use background-image make sure you wrap the url with quotes.