P
ApplePieGiraffe• 30,565
@ApplePieGiraffe
Posted
Hey, Luke! 👋
Congratulations upon completing your first Frontend Mentor challenge! 👏
Good work on this one, too! 👍
I suggest,
- Using CSS background images to add and place the background SVGs rather than adding directly into your HTML. Here's an article on CSS background images to get you started, if you like.
- Not using the
<table>element for the footer of the card since it isn't really a table and table layouts aren't used very much nowadays. Flexbox would a more modern, clean way to layout the footer very easily! - Making the
box-shadowaround the card component a little subtler would be a nice touch. 😉
Keep coding (and happy coding, too)! 😁
1
Luke• 220
@lukebergmann
Posted
Incredible! Thank you for your critique. I appreciate the help and will definitely put it into action. I need to become more acquainted with flexbox thats for sure!
0
