@vanzasetia
Posted
Hi Jean! π
Congratulations on finishing this challenge! π
I recommend removing all the <div>
elements. You do not need to wrap every element with a <div>
. For example, you can make the <img>
as a block element (display: block
) and set max-width: 100%
to prevent the image from overflowing.
For the alternative text, I recommend telling "what the image is". In this case, it is a QR code that will redirect the users to frontendmentor.io. Also, you do not need to write the word "image" on the alternative text. Users will already know that it is an image. I recommend reading Quick tip: Using alt text properly - The A11Y Project.
I hope this helps. Happy coding! π
Marked as helpful
@Jwalt95
Posted
Thank you very much for the tips@vanzasetia