Hello Jhon! Your solution looks neat and organized, especially with the use of BEM. It matches very well with the original design. As for the index.html file, you seem to have used the <svg>
tag for displaying illustration-article.svg. This tag can be helpful if you want to customize the svg directly from the editor, but you may also want to use the <img>
tag instead for simplicity purposes, like this:
<img src="assets/images/illustration-article.svg">
Other than that, great job!
Marked as helpful
@vincentwilliamrodriguez
Hi dude, thank you so much for the feedback, I really appreciate it since it means a lot to me.
The reason that I didn't use the <img>
was exactly I wanted to change its design using CSS, to be precise: Its dimensions.
However; it's not a bad suggestion, in fact, it's very good.