@beowulf1958
Posted
Great job completing the challenge. Your page is awesome so far; however, I did notice some problems in the html.
First, all img tags need an alt value. This helps accessibility, and serves as a backup if the image does not load for any reason. Something like alt="bottle of Chanel perfume". Second, there are no h1 tags, and you skip from h2 to h6. Check out the article at Headings for more information.
The next step is to make the page responsive so it can be seen on a smaller (375px) screen. If you use @danielmrz-dev advice on the <picture> tag, you make the page more semantic and can swap out the desktop image with the mobile image with a @media query
/* mobile version */
@media screen and (max-width: 480px) {
.grid {
grid-template-columns: 1fr;
}
}
I was able to reverse-engineer a responsive version of your page that way. You might have to adjust some font-sizes, etc, but a relatively simple fix.
Have a great day, and Keep on Coding!