Tereza• 605
@sirriah
Posted
Hello, nice work! :)
I have some suggestions:
-
the .purple div is unnecessary. You can add
background: hsl(277, 64%, 61%);to the div .image and to the .img-desktop addmix-blend-mode: multiply;. Result be the same and there will not be the problem with 101%. -
You don't have to use the double img tag for desktop and mobile version. Try to use
<img srcset="...">- link to the documentation. -
try to use BEM naming in you future projects. This is small project, but naming div "image" is not the best practise in a bigger project.
0
Guilherme• 20
@glhrmarques
Posted
@sirriah, thank you so much for your contribution! I'll make sure to learn all your suggestions.
0
