@vanzasetia
Posted
Hi, Emanuel Bonardo! π
Here are some suggestions for improvements:
- No
<div>
elements: Remove all the<div>
elements. The styling of the.container
and the.card__container
can be merged and put on the<main>
element. You do not need to wrap each element with a<div>
element. - Use owl selector: For spacing between elements inside the card, you can use the owl selector (
* + *
). For example, you can have.card > * + * { margin-top: 1em }
. - Alternative text should describe the image: Alternative text for images should not include any words that are related to the word "image". The semantic meaning of the
<img>
element tells assistive technologies to pronounce it as an image. So, you should describe what is the image and the purpose of the QR code.
I hope this helps. Happy coding! π
Marked as helpful