@danielmrz-dev
Posted
Hello @LadyLucyfurr!
Your solution looks great!
I have just one suggestion for improvement:
- Since this project is not responsive (mobile and desktop version are pretty much the same), you can change just one part of your code to make your project fit for both mobile and desktop:
You set width: 85vw;
. This will make your card occupy 85% of the viewport width on all screen sizes. That's why it gets so big on desktop version.
To fix that, you can set a fixed width
value, like 360px
, for example. By doing that, your card will have a fixed width and won't overgrow on bigger screens. In this case, using fixed values like px
is not a problem, because like I said before, desktop and mobile versions are the same.
I hope it helps!
And happy new year! 😊
Marked as helpful
@teenabobeena
Posted
@danielmrz-dev I have updated the width to px. Thank you so much for your valuable feedback!