Michael Bishop• 1,070
@MikeBish13
Posted
Great job on this project!
One small point: you've used a <p> tag to create your image overlay, which is bad practice when we talk about semantic HTML and how browsers, search engines, screen readers, etc interpret your page. A <div> would probably be better suited here.
Better still, you could maybe use a pseudo element to create the image overlay which could also hold the image in the content property - that way you don't need to use 2 elements and have 2 hover selectors.
Give it a try! Best of luck!
Marked as helpful
2
