@DylandeBruijn
Posted
Hiya @rafe-designs,
Welcome to Frontend Mentor and congratulations on your first solution!
I'm happy to give you some friendly constructive feedback:
-
At the moment your card is pretty small which makes the QR code hard to scan. I suggest removing the
height
andwidth
restrictions on theimg
and putting amax-width: 100%
on there. Also it seems the image you use is different from the one provided by the challenge. I suggest taking another look at the challenge briefing. -
Try using more descriptive classes like
.card
and.card-title
. This makes it easier for you to style your elements and for other developers to read your code. -
You could try using an
article
element for you card instead of adiv
to give it more semantic meaning. -
Try putting the
font-family
on thebody
element instead of your card. This way you make use of the cascade. In other words all children of thebody
will inherit thefont-family
of thebody
. You can always overwrite thefont-family
for specific elements by setting it explicitly! -
CSS classes by convention are in lowercase like so
.qr-code-card
, instead ofQr-code-card
.
I hope you find my feedback helpful, as always I would appreciate it if you could mark my comment as helpful when it was!
Let me know if you have more questions and I'll try to answer them.
Marked as helpful