@luqmanx1998
Posted
Hey there, I like your solution ! You have a good grasp on a lot of important CSS properties, and even using custom CSS variables, which is great. I'm also learning, but it's great to see someone else do their best ! Here's my feedback after looking at your code :
- Perhaps in the future it's best not to explicitly set your container's height, as it could affect your card layout, as seen when in desktop mode, your card content is stretched out. In your
.card
container you had setheight: 80vh
. Because your content did not fill up the entire height, combined with the property you had set.card-body { flex: 1 1 auto; }
caused your content to stretch out. If you remove both these properties the card collapses, and the layout looks more even !
That being said, your card looks great, and let's keep improving :D
Marked as helpful