Arne• 1,110
@Dudeldups
Posted
Heya! This looks good! Two things I saw:
- For the hover state of the image, you can use the
::afterpseudo element with
background: #color url(eye-image) center no-repeat;
And then use opacity transition to show or hide it on hover. This way, the "eye" icon also transitions :)
- You used way too many
flexcontainers. For the div where the price and remaining days are shown, you can put all of that in a div, give it agapand usemargin-right: autoon the price. This way, it pushes the other elements all the way to the right :)
Marked as helpful
0
ThanHtike（テンタイ）• 110
@CGM-ThanHtike
Posted
Thank you for the feedback @Dudeldups. I just fixed with opacity and it looks way better. And for the margin-right: auto method, I was try that before but the problem is, I was struggling with aligning icons and text vertically (that's why I used flex also for it).
0
