thomasbromehead / colmar Goto Github PK

Most of your HTML looks good, but make sure to keep it all neat by properly nesting! You have a couple of divs out of place here.

I noticed you have "margin: 0" in a lot of different sections, try using a universal selector (*) for this!

I'm not sure if this worked on your computer, but be sure to properly link your CSS file. You forgot to add the './' at the beginning of your href, so when I load your code, it shows up with no styling. This is how it should be:

<link href="./Resources/style.css" rel="stylesheet" type="text/css"/>

Good job with this project! You had a couple of minor issues with line breaks, HTML nesting, and linking your CSS file, but your code is effective and clear! I'd mostly recommend going through your code and organizing it better; a little bit of organization goes a long way in making your code look professional.

Just a minor issue, but try to keep your code neat by eliminating these unnecessary line breaks

Hello team,
I successfully removed the man reading the journal along with the description underneath it by giving the div with class "img-holder" an id of "journal".
However I left it on so you give it a try with the nth-of-type selector.
I tried it but it also removes the image at the bottom.
There are a total of 3 img-holder divs...
Let me know if you manage to remove if from the DOM in that way, I would really like it to work in that way.
Thanks a lot, the courses are great, loved every bit of it!!