Hello dyl25,
This is a really good solution. I would like to suggest that you use .container
class on the main element instead of creating another element (div).
Also, wrap img in a container like 'image-container' class. This is required for uniform rendering on some versions of internet explorer. It is also considered a best practice.
img element must have attribute alt="description of the image". This is displayed if the image could not be loaded. However, if the image intended to be used just for design, empty string should be used instead. For example, <img src="my-image.png" alt="my profile picture"> or <img src="background-image.png" alt="">. This is also good for accessibility.
I hope you find it useful.
Marked as helpful
@dyl25
Posted
@Fasunle
Hello,
Thanks for your feeback! I will take it into account for my next try and next challenges.
@dyl25 It is my pleasure.