@correlucas
Posted
Hello Ian Rioba, congratulations for your solution!
Your solution is working really good and I've not much to say about it in general.
But if you want to clean your code, even if is already working, you can build this component using a single div to hold all the content (img, h1 and p)
and clean all the other divs.
<body>
<article>
<img>
<h1></h1>
<p></p>
</article>
</body>
Note that also for a clean css in this case you don't need any class, you can use each elements selectors to manage it in the css sheet, for example article, img, h1 and
can be used with no need for classes.
Doing that you'll have a lean and clean solution and write 20% less code than the current html/css sheet.
Hope it helps and happy coding!
@Rioba-Ian
Posted
@correlucas Thank you for your review. You are absolutely right. I shouldn't introduce a div to hold the
<h1> and <p> inside the article. I ended up increasing CSS. I will review and make those changes.