@mendezpvi
Posted
Excellent work! 👏, I have some suggestions:
- Use semantic tags. For example, the class
qr-wrapper
can be anarticle
orsection
. - I consider the tags of the
container
andqrtext
classes to be unnecessary. - The
text-align
andpadding
properties can go in theqr-wrapper
class. - The requirements of this project indicate that the media query is not necessary, the card does not change.
- The properties you gave to the
wrapper
class would be better given to thebody
, changing theheight
tomin-height: 100vh
. Then you apply apadding-block
ormargin-block
tofooter
of your discretion. Ex. 1rem. - You forgot to change the
font
, and thebackground
doesn't match. Every detail counts.
I invite you to visit my solution here
Happy coding! 😎