Renato Lacerda• 160
@ralacerda
Posted
Hey, awesome job!
You got everything right and your code is really clean.
People usually show the attribution at the bottom (but I can understand if you hide it to keep the design clean). You can try using position-absolute to place it at the bottom without moving the card.
If you want to fix the accessibility problems you can use a <main> tag instead of a <div> in your box element. It should fix both warnings about landmarks.
You can include an empty alt attribute to your img tag, or describe that it is a qr code.
Marked as helpful
1
LEANDRO• 50
@JESUS-LEANDRO
Posted
@ralacerda obrigado pelas dicas irei bota em pratica.
2
Ahmed Ramadan• 80
@ahmed-abohmaid
Posted
@ralacerda Thanks, I will make these changes.
0
