@Dytoma
Posted
Hey 👋
Good job on completing this challenge, your solution looks amazing👌
However I do have one suggestion when I looked at your code:
- Instead of using a
<div class="line"></div>
I suggest you useborder-bottom
as the div add more content to your page, it is better to useborders
.
Happy coding👏
Marked as helpful
@Bakhriev
Posted
@Dytoma thanks for the feedback,in this example, should the background images be done with the img tag? or should they be done with the background?
@Dytoma
Posted
@Bakhriev you're welcome🙌
For the call to action section you can use the background property to give it both the background color
and the background image
but for the other images, you can use the img
element and wrap it in a div so that you can position it relative to the div
.
You can read more about backgrounds
and how to set it on MDN multiple background docs.
Marked as helpful