@GabrielBC2021
Posted
Hello :D
1.- You can eliminate the default HTML that comes in the projects, so your website looks a little bit better, just a little detail.
2.- You can eliminate the next piece of code, that is not neccesary
.preview-card__image-container:hover .preview-card__image { opacity: 0.3; }
3.- When you hover the image, the color coves the background image and in the design is not full cover, so you can just modify that opacity like this:
.preview-card__image-container--overlay:hover { opacity: 0.5; background-color: var(--currency-color); cursor: pointer; }
Just some details that will make your website looks better. Good work, keep it up :)
Marked as helpful
@leslief10
Posted
@GabrielBC2021 Hi thanks for the feedback! I will keep your first point in mind and delete the code you mentioned on the second point. Regarding your third point, I decided to leave the opacity at 0.9 because the eye icon inside this container would stop being white with a lower opacity. Again, thank you so much for your recommendations!