P
Daniel Clement• 80
@DanielClement37
Posted
On the html side of things you could have been more descriptive with which html tags you used and named some of the classes better to be more descriptive. For example the article image is wrapped by a div named <div class="img">. That can get confusing when on the css side of things if you type .img{ /*css here */} vs just img{ /*css here */} and be effecting entirely different elements. For a naming suggestion instead of the <div class="img"> I used this
<figure class="blog-art">
<img src="./assets/images/illustration-article.svg" alt="blog-art" />
</figure>
Over all though your solution visually is very close to the design and the only critique for that I would say is that it is slightly bigger.
0
