I am glad to see you using semantic HTML, as it’s a point I do tend to bring up often. The effective use of the
figure tags, and the
header tag used appropriately… very nice!
One thing to note: you have a
<div id='img-div'> wrapping the
figure at the start of your
main tag. But that figure is the only thing in that div, which makes the div completely unnecessary. Instead, apply the
id='img-div' to the figure itself, it’ll still pass the test and get rid of pointless elements.
Now, I would suggest you consider continuing that same semantic thing. I would say that you don’t need
div tags in your code at all - instead, maybe create an
article tag with the
id='tribute-info', and each
h4 tag (and it’s accompanying list) could be wrapped in a
section tag. That makes sense - if you think about it as, for example, a magazine article, each unique block of information becomes a section.
Using that approach,
id='brands' simply becomes another section within that
article, keeping everything tidy and organized in a logical structure.
All that said, it passes all the tests. Let me be the first to congratulate you on a very well-constructed, and attractive, tribute page!
Down the line, as your coding skills become more refined, you may want to consider revisiting this and other lessons, with an eye toward “how can I beautify/simplify/change this?” It’s a common practice, and lets you demonstrate the evolution of your skills.
Again, very very nicely done!