Separating your HTML

[jema28]

For this project I would consider keeping all the HTML in the index.html. The portfolio-card.html doesn't seem to have content (especially the img tag):

I would move this into your index.html and make sure you include content in the tags :)

[jessicasalmon]

second @jema28. I really love that you've thought about how you could split your code up and make it reusable. This way of thinking is going to come in really handy when you start using templating engines and frontend frameworks.

It feels like a potential overcomplication for this use case, where you're never going to have more than a few cards at a time.

Also please ensure that you delete any redundant files. e.g card.html has no content