Éric Férole• 420
@Eric-Ferole
Posted
Good job Claire. It's close to perfect.
Here some recommendations :
- Add a border radius to .qr-background of 10px.
- For your QR image and background, to make things simpler, you could add the background directly to the div.qr-code and align your <img qr-code.svg> in the center of the div with flexbox.
- Media query is not needed here because the size of the div.card in desktop resolution is the same as in mobile resolution.
- Review your accessibility issues and make sure you understand them.
Keep up and see you around ;)
Éric F.
1
